You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'd like to conduct a software engineering experiment regarding the benefit of Pylint alerts removal.
The experiment is described here.
In the experiments, Pylint is used with some specific alerts, files are selected for intervention and control.
After the interventions are done, one can wait and examine the results.
I'm asking for your approval for conducting an intervention in your repository.
@bl4de , I created a PR that fixes the alerts.
Change are rather small.
Please see if it is OK.
Before merging, I'd like to discuss you on two more alerts.
In the file nodestructor\nodestructor.py there is an broad-exception-caught on line 382
A broad exception might hide unexpected exception types and therefore it is recommended to catch specific exception.
I could not understand which exceptions are expected there.
Can you consult me?
Also, in the file jwt_decoder.py there are too long lines.
Ussally adding a new line solves tat but these lines are:
I'd like to conduct a software engineering experiment regarding the benefit of Pylint alerts removal.
The experiment is described here.
In the experiments, Pylint is used with some specific alerts, files are selected for intervention and control.
After the interventions are done, one can wait and examine the results.
I'm asking for your approval for conducting an intervention in your repository.
See examples of interventions in stanford-oval/storm, gabfl/vault, and coreruleset/coreruleset.
You can see the planed interventions
@bl4de , may I do the interventions?
The text was updated successfully, but these errors were encountered: