Separate pylint run for files under angr/procedures

angr / ci-settings

Docker image and azure templates for angr's CI

2 stars 6 forks source link

Separate pylint run for files under angr/procedures #22

Open rhelmot opened 3 years ago

rhelmot commented 3 years ago

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 commented 3 years ago

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 commented 2 years ago

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.