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

Separate pylint run for files under angr/procedures #22

Open
rhelmot opened this issue Jul 7, 2021 · 2 comments
Open

Separate pylint run for files under angr/procedures #22

rhelmot opened this issue Jul 7, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@rhelmot
Copy link
Member

rhelmot commented Jul 7, 2021

We should have a separate pylintrc and pylint process for files under angr/procedures. Specifically, we should disable arguments-differ,unused-argument,no-self-use,missing-module-docstring,attribute-defined-outside-init while linting simprocedures.

@twizmwazin
Copy link
Member

Ideally we should see if there is a way to do this in a single pylintrc so that the local setup wouldn't be complicated by a change like this. Even if we were to provide a script for manual linting, that wouldn't automatically translate well into editor configurations.

@twizmwazin
Copy link
Member

twizmwazin commented Dec 1, 2021

An update on this, I've looked to see if pylint can support separate rules natively. Currently it cannot, but efforts to fix that are being tracked here, though I'm not sure what the rate of progress is.

@twizmwazin twizmwazin added the enhancement New feature or request label May 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants