-
-
Notifications
You must be signed in to change notification settings - Fork 812
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
ADMIN: Fix the eslint_disable_check.py
script
#2257
Comments
@palisadoes I fixed the issue in API. Can you please assign it to me as i would fix it in admin also |
This issue did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue. |
It's assigned to you |
@palisadoes Can you please close the issue since the pr is merged and i can be assigned other issues |
Describe the bug
eslint_disable_check.py
script to detect the use of theeslint-disable
string in files submitted in the PR// eslint-disable-next-line @typescript-eslint/no-unused-vars
To Reproduce
Steps to reproduce the behavior:
// eslint-disable-next-line @typescript-eslint/no-unused-vars
Expected behavior
// eslint-disable-next-line @typescript-eslint/no-unused-vars
The script must fail with the string
eslint-disable
discovered any where in any lineActual behavior
Screenshots
Additional details
The same person should be assigned to both these issues
eslint_disable_check.py
script talawa-api#2519eslint_disable_check.py
script #2257Potential internship candidates
Please read this if you are planning to apply for a Palisadoes Foundation internship
The text was updated successfully, but these errors were encountered: