Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

With the latest release --fix defaults to none instead of all #3771

Closed
audgirka opened this issue Sep 27, 2023 · 0 comments · Fixed by #3777
Closed

With the latest release --fix defaults to none instead of all #3771

audgirka opened this issue Sep 27, 2023 · 0 comments · Fixed by #3777
Assignees
@audgirka
Labels
bug

Comments

@audgirka
Copy link
Contributor

audgirka commented Sep 27, 2023 •
edited
Loading

After the latest release where we replaced --write with --fix the transform is not invoked, if I run the following command:
ansible-lint examples/playbooks/transform-no-handler.yml --fix

Always need to pass argument to the --fix like all or <rule name>

ansible-lint examples/playbooks/transform-no-handler.yml --fix no-handler

I think it is coming from this change: https://github.com/ansible/ansible-lint/pull/3751/files#diff-7dc48b0a7a98961bd73a823b91640268a2a62efc005a2f5bda43cdb834dbed87R281
@audgirka audgirka added new Triage required bug labels Sep 27, 2023
@ssbarnea ssbarnea self-assigned this Sep 27, 2023
@ssbarnea ssbarnea removed the new Triage required label Sep 27, 2023
@audgirka audgirka assigned audgirka and unassigned ssbarnea Sep 28, 2023
@audgirka audgirka mentioned this issue Sep 28, 2023
Merged
@github-project-automation github-project-automation bot moved this from Roadmap to Done in 🧰 devtools project board Sep 28, 2023
@cognifloyd cognifloyd moved this from Backlog to Done in ansible-lint --fix Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@audgirka audgirka

Labels
bug
Projects
ansible-lint --fix
Status: Done
🧰 devtools project board
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

--fix will now default to all ansible/ansible-lint
2 participants
@ssbarnea @audgirka