Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix pre-commit errors #80

Merged
merged 2 commits into from
Mar 22, 2024
Merged

Fix pre-commit errors #80

merged 2 commits into from
Mar 22, 2024

Commits on Mar 21, 2024

  1. Modify a docstring to avoid exceeding line length limit 

    A long line in this docstring was causing the flake8 pre-commit check to
fail, so I edited the comment to be compliant. I shortened the lines to
about 80 characters for consistency with other comments, even though
that's more of a change than is strictly required to pass flake8. With
this change, we can start requiring pre-commit to pass for all merges.

I also changed a note to use Restructured Text syntax, for compatibility
with other docstrings in the project.
    @diazona
    diazona committed Mar 21, 2024
    Configuration menu
    Copy the full SHA
    eee3973 View commit details
    Browse the repository at this point in the history

  2. Skip actionlint pre-commit hook in CI 

    The actionlint hook uses Docker, which isn't configured in the runners
used by pre-commit.ci, so I'm marking that one to be skipped. In
the future we can set up some alternate way of running actionlint on
pull requests automatically.
    @diazona
    diazona committed Mar 21, 2024
    Configuration menu
    Copy the full SHA
    0ce2310 View commit details
    Browse the repository at this point in the history