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 Select Parameter #144

Closed
kornpow opened this issue Mar 30, 2022 · 3 comments
Closed

Allow Select Parameter #144

kornpow opened this issue Mar 30, 2022 · 3 comments

Comments

@kornpow
Copy link

kornpow commented Mar 30, 2022

For example:
flake8 --ignore F401,F541,F811,F841,E,W --select E999 --max-line-length 100 .

I want ignore all E rules, except for what is shown in select

@chrissimpkins
Copy link
Contributor

chrissimpkins commented Apr 1, 2022

Possible to give this a shot with the args definition in your configuration?

https://github.com/py-actions/flake8#args

The intent is to support any argument that you can use with flake8. So, I think that you should be able to configure it with the current release of the Action. The exclude and ignore are only intended as shortcuts to commonly used arguments. You can define the entire set of arguments with args instead. And you can combine args with args that are not explicitly defined by the others.

But this doesn't fully get at your issue as I understand. We can certainly consider an explicit definition for this argument similar to the exclude and ignore inputs. Do you feel that this is used commonly and warrants its own input field so that it can be used with the ignore field?

@kornpow
Copy link
Author

kornpow commented Apr 1, 2022

Im sorry, I mustve glossed over that while reading the docs! args worked for me

@kornpow kornpow closed this as completed Apr 1, 2022
@chrissimpkins
Copy link
Contributor

Great! Let me know if you run into any issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants