You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This should essentially just require that we switch our linter config to universally enforce 80 chars/line, and add the # noqa: E501 tag to all docstrings > 80 chars wide.
The text was updated successfully, but these errors were encountered:
For long lines of URLs in the docstrings, we should tag them with # noqa: E501.
For other types of long lines that can be adjusted, we should just adjust them. They exist in the codebase because we did not enforce 80 char for a while. So some PRs with other long lines also got merged.
The PR should also change the max-line-length to 80 from 200 in setup.cfg.
See conversation in #1509.
This should essentially just require that we switch our linter config to universally enforce 80 chars/line, and add the
# noqa: E501
tag to all docstrings > 80 chars wide.The text was updated successfully, but these errors were encountered: