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 #98

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

Pylint alerts corrections as part of an intervention experiment #98

evidencebp opened this issue Nov 24, 2024 · 2 comments

Comments

@evidencebp
Copy link

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

Since the plan includes only 3 internevtions, I'll submit a PR fixing them.
I'll be happy to know what do you think of the PR and if you would like to modify it or merge it, @akrherz

@evidencebp
Copy link
Author

This is the PR

@evidencebp
Copy link
Author

I see that the CI/CD alerts on commented out code.
This comment was in the original file, its position was changed due to method extraction.
@akrherz , please tell me if you want me to remove it.

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

1 participant