-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Fix validation error type GL04
and check in CI
#25106
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
Comments
GL04
from code baseGL04
and check in CI
Am working on this. :) |
@MarckK : i'm not 100% certain but I think you'll be able to just use DataFrame and/or Series in place of NDFrame now. Also, it may not be worth fixing the errors on the Panel method docstrings. is there a way to skip the GL04 check? |
Hi @simonjayhawkins, the GL04 check is not currently part of CI. A number of checks were suggested as additions and are slowly being added in. I'll just leave this to the side for now. |
GL04 is currently checked in CI so I think this can be closed |
Problem description
In order to have a continuous check by the CI on validation error
GL04
(Private classes should not be mentioned in public docstrings), fixing them in the code base enables the addition to the CI for automated testing in the future.GL04
has just 32 occurrences (see overview of the errors):Todo:
code_check.sh
script to take into account theGL04
type of errorsThe text was updated successfully, but these errors were encountered: