Skip to content
New issue

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

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

Already on GitHub? # to your account

Crash since 24.5.0: 1 MatchError called incorrectly as line numbers start with 1 #4177

Closed
nvuillam opened this issue May 19, 2024 · 3 comments · Fixed by #4195
Closed

Crash since 24.5.0: 1 MatchError called incorrectly as line numbers start with 1 #4177

nvuillam opened this issue May 19, 2024 · 3 comments · Fixed by #4195
Assignees
Labels

Comments

@nvuillam
Copy link

Summary

ansible-lint returns the following error
MatchError called incorrectly as line numbers start with 1

Issue Type
  • Bug Report
OS / ENVIRONMENT
ansible-lint 24.5.0 using ansible-core:2.16.6 ansible-compat:24.5.1 ruamel-yaml:0.18.6 ruamel-yaml-clib:0.2.8
  • ansible installation method: pip
  • ansible-lint installation method: pip
STEPS TO REPRODUCE
ansible-lint
1 MatchError called incorrectly as line numbers start with 1
Desired Behavior

No crash :)

Actual Behavior

crash with 1 MatchError called incorrectly as line numbers start with 1

You can see the error in MegaLinter build log: https://github.com/oxsecurity/megalinter/actions/runs/9149776695/job/25153984448?pr=3559

@nvuillam nvuillam added bug new Triage required labels May 19, 2024
nvuillam added a commit to oxsecurity/megalinter that referenced this issue May 19, 2024
nvuillam added a commit to oxsecurity/megalinter that referenced this issue May 19, 2024
* [automation] Auto-update linters version, help and documentation

* [MegaLinter] Apply linters fixes

* Downgrade lightning flow scanner

Until bug is fixed: https://github.com/Lightning-Flow-Scanner/lightning-flow-scanner-vsce/issues/100

* Downgrade ansible-lint to v24.2.3

Related to ansible/ansible-lint#4177

* fix pylint issues

* [MegaLinter] Apply linters fixes

---------

Co-authored-by: nvuillam <nvuillam@users.noreply.github.com>
@ssbarnea
Copy link
Member

@nvuillam Unless you can update the description to include a reproducer, we will need to to close thing bug.

@nvuillam
Copy link
Author

@ssbarnea the test files are located here (and they are the same for ages :) )

https://github.com/oxsecurity/megalinter/tree/main/.automation/test/ansible/bad/ansible

Does it help ?

@nvuillam
Copy link
Author

nvuillam commented Jun 2, 2024

@ssbarnea thanks !
Do you know when it will be released in latest version ? :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants