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.

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

Allow to specify command line argument of actionlint #1852

Closed
rhysd opened this issue Aug 10, 2021 · 13 comments · Fixed by #3772
Closed

Allow to specify command line argument of actionlint #1852

rhysd opened this issue Aug 10, 2021 · 13 comments · Fixed by #3772
Labels
enhancement New feature or request O: backlog 🤖 Backlog, stale ignores this label

Comments

@rhysd
Copy link
Contributor

rhysd commented Aug 10, 2021

Is your feature request related to a problem? Please describe.

actionlint provides -ignore option to ignore some errors. This is necessary to ignore some false positives or errors which a user is not interested in. However, super-linter currently does not provide a way to pass command line options to actionlint.

Related issue: rhysd/actionlint#26

Describe the solution you'd like

I'd like to have the way to ignore some errors from actionlint. It'd be nice to have some environment variable for it.

Describe alternatives you've considered

N/A

Additional context

N/A

@github-actions
Copy link
Contributor

github-actions bot commented Sep 9, 2021

This issue has been automatically marked as stale because it has not had recent activity.
It will be closed in 14 days if no further activity occurs.
Thank you for your contributions.

If you think this issue should stay open, please remove the O: stale 🤖 label or comment on the issue.

@github-actions github-actions bot added the O: stale 🤖 Stale issue/pr label Sep 9, 2021
@rhysd
Copy link
Contributor Author

rhysd commented Sep 23, 2021

This issue is stuck because of #1667

@github-actions github-actions bot removed the O: stale 🤖 Stale issue/pr label Sep 23, 2021
@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity.
It will be closed in 14 days if no further activity occurs.
Thank you for your contributions.

If you think this issue should stay open, please remove the O: stale 🤖 label or comment on the issue.

@github-actions github-actions bot added the O: stale 🤖 Stale issue/pr label Oct 24, 2021
@github-actions github-actions bot closed this as completed Nov 8, 2021
@ferrarimarco
Copy link
Collaborator

Reopening this one, given that #1667 was abandoned.

@ferrarimarco ferrarimarco reopened this Nov 23, 2021
@ferrarimarco ferrarimarco removed the O: stale 🤖 Stale issue/pr label Nov 23, 2021
@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity.
It will be closed in 14 days if no further activity occurs.
Thank you for your contributions.

If you think this issue should stay open, please remove the O: stale 🤖 label or comment on the issue.

If you're a maintainer, you can stop the bot to mark this issue as stale in the future by adding the O: backlog 🤖 label`.

@github-actions github-actions bot added the O: stale 🤖 Stale issue/pr label Dec 24, 2021
@ferrarimarco ferrarimarco added O: backlog 🤖 Backlog, stale ignores this label and removed O: stale 🤖 Stale issue/pr labels Dec 24, 2021
@vorotech
Copy link

vorotech commented Jan 5, 2022

One more case example. Atm there is no way to ignore an error which occur when using workflow_call
The secrets.GITHUB_TOKEN causes a false positive

property "github_token" is not defined in object type

See rhysd/actionlint#87

@ferrarimarco
Copy link
Collaborator

@vorotech at least this one should be fixed in the next super-linter release that ships with the latest actionlint release.

@SoumayaMauthoorMOJ
Copy link

Any update on this issue? This would allow me to ignore the 'SC2016:' error which is linked with https://github.com/github/super-linter/issues/3100

@artempas
Copy link

can't wait to see it

@ferrarimarco
Copy link
Collaborator

There's some work already done in #1853. I'll have a look into that to recover what @rhysd did.

@rhysd
Copy link
Contributor Author

rhysd commented Oct 16, 2022

@ferrarimarco I recreated a PR onto the latest main branch at #3451.

@rhysd
Copy link
Contributor Author

rhysd commented Dec 3, 2022

Closing without fixed since the maintainer is not interested in this issue.

@ifyum
Copy link

ifyum commented Jan 6, 2023

ya arreglaron esto ? Run psql -h localhost -p 4[9]
psql: warning: extra command-line argument "pass" ignored
Password:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment