treewide: apply linting #1299
treewide: apply linting #1299
Conversation
PolynomialDivision
force-pushed
the
apply-linting
branch
2 times, most recently
from
503e2f3 to
9192bd4
Compare
| ipv6_prefix: 2001:bf7:820:3300::/56 | ||
|
|
||
| networks: | ||
|
|
There was a problem hiding this comment.
Wollen wir diese Leerzeilen-Regel nicht lieber deaktivieren? Ich arbeite sehr gerne mit Leerzeilen um mich beim Scrollen einfacher orientieren zu können, und ich finds auch wichtig ein bisschen Freiheit zu behalten, da wo es funktional keinerlei Unterschied macht.
Und ich verstehe auch nicht so ganz, warum sechs Zeilen weiter (eine Objektebene tiefer) die Leerzeilen scheinbar okay sind ;)
| changed_when: false | ||
| command: scp -O -o StrictHostKeyChecking=no {{ image }} {{ ansible_user }}@[{{ ansible_ssh_host }}]:/tmp/ | ||
|
|
||
| ansible.builtin.command: scp -O -o StrictHostKeyChecking=no {{ image }} {{ ansible_user }}@[{{ ansible_ssh_host }}]:/tmp/ |
There was a problem hiding this comment.
Diese Leerzeile war schon okay dort
PolynomialDivision
force-pushed
the
apply-linting
branch
from
9192bd4 to
7cd5192
Compare
Noki
left a comment
Noki
left a comment
There was a problem hiding this comment.
besides the two locations that are already deleted in the main branch, and the comments about empty lines from pktpls that needs to be resolved, this is good to go. Personally I have no real preference about the empty lines pointed out by pktpls.
PolynomialDivision
force-pushed
the
apply-linting
branch
2 times, most recently
from
e35eb50 to
c5af282
Compare
Use ansible-lint --fix.
Use prettier --write "**/*.yml" to apply linting.
Updated all builtin module actions to use fully qualified collection names (FQCN) as required by ansible-lint rule fqcn[action-core]. Command used: ansible-lint --fix This improves compatibility with Ansible 2.10+ and aligns with best practices.
PolynomialDivision
force-pushed
the
apply-linting
branch
from
c5af282 to
f219c53
Compare
Running ansible-lint --fix.
Running ansible-lint --fix.
Running ansible-lint --fix.
|
Der PR ist hier so lange in der Schwebe, weil es um gundsätzlich um Linting geht. Wenn man anfängt, Warnungen nach und nach auszuschalten, verliert das Linting für mich irgendwie seinen Sinn. Ich brauche da ehrlich gesagt keine Freiheit, ich will einfach nur meine Tastenkombination im Editor drücken, damit die Datei gelinted und formatiert wird, und außerdem einen einheitlichen Stil über alle Targets hinweg durchsetzen. Mir ist es völlig egal, ob da nur eine Leerzeile ist oder nicht, so lange es einheitlich ist, und ich nen linter benutzen kann. Wenn Leute da aber gern mehr Freiheit wollen, dann bin ich damit auch fein. Aber das beißt sich dann damit, dass ich gerne meine Files automatisch gelinted haben will. |
|
Ich weiß nicht genau wie ich in der ansible-lint skip liste dieses Leerzeile Linting rausnehme. |
No description provided.