8000 Pylint alerts corrections as part of intervention experiment · Issue #3835 · coreruleset/coreruleset · GitHub 8000
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
Pylint alerts corrections as part of intervention experiment #3835
Closed
@evidencebp

Description

@evidencebp

Motivation

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 the interventions in your repositories.The interventions are expected to benefit the project and will be submitted in PR for approval.

These are the planed interventions.

@airween , I noted the you wrote util/crs-rules-check/rules-check.py
May I fix the alerts?

Proposed solution

In the case of the broad-exception-caught, it should be replaced with a more exact one.

Alternatives

Code might be kept as is.
This is a suggested (small) improvment.

Additional context

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0