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

Pylint alerts corrections as part of an intervention experiment #21

Open
evidencebp opened this issue Nov 24, 2024 · 2 comments
Open

Pylint alerts corrections as part of an intervention experiment #21

evidencebp opened this issue Nov 24, 2024 · 2 comments

Comments

@evidencebp
Copy link
Contributor

Pylint alerts are correlated with tendency to bugs and harder maintenance.
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.

Your repository is expected to benefit from the interventions.
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
It includes just two broad-exception-caught alerts.
One was simple so I fixed it and I'll submit a PR.
If you'll be able to consult me regarding the other, it will be great.

I'll be happy to know what do you think of the PR and if you would like to modify it or merge it, @kianby

@kianby
Copy link
Owner

kianby commented Nov 24, 2024

I'm glad to participate in this experiment. I'm fine with PR #22.

What should be the second fix?

@evidencebp
Copy link
Contributor Author

Thanks a lot, @kianby!
src\stacosys\run.py catches Exception in line 103.
The try section just calls main.
The function load_and_validate_config might raise FileNotFoundError or ValueError.
However, I could not know for sure which other exceptions are might be neede here.

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