Skip to content

GitHub Action: Check Code Coverage Bypass Statements and Eslint Disable Statements #2806

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

Closed
im-vedant opened this issue Dec 29, 2024 · 2 comments
Assignees
Labels
feature request test Testing application

Comments

@im-vedant
Copy link

Is your feature request related to a problem? Please describe.
Yes, the current workflow does not automatically detect or flag instances where:

  1. Code coverage is bypassed using comments like /* istanbul ignore next / or / istanbul ignore file */.
  2. ESLint rules are disabled using comments like /* eslint-disable / or / eslint-disable-next-line */.

These statements can bypass essential checks and affect code quality or test coverage reporting.

Describe the solution you'd like
Implement a GitHub Action that:

  1. Scans for Code Coverage Bypass Comments:
    Detects /* istanbul ignore next /, / istanbul ignore file */, and other related comments in PRs.
    Fails the workflow if such comments are detected without proper justification.
  2. Scans for ESLint Disable Comments:
    Detects /* eslint-disable /, / eslint-disable-next-line */, and similar comments in the code.
    Ensures that any disabled rule is logged and flagged for manual review.
@im-vedant
Copy link
Author

@palisadoes Can you assign me this issue?

Copy link

Congratulations on making your first Issue! 🎊 If you haven't already, check out our Contributing Guidelines and Issue Reporting Guidelines to ensure that you are following our guidelines for contributing and making issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request test Testing application
Projects
Archived in project
Development

No branches or pull requests

2 participants