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.

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

Autoupdate pre-commit hooks #603

Merged
merged 2 commits into from Feb 20, 2023
Merged

Autoupdate pre-commit hooks #603

merged 2 commits into from Feb 20, 2023

Conversation

felddy
Copy link
Owner

@felddy felddy commented Feb 20, 2023

πŸ—£ Description

Fixes broken isort pre-commit hook.
Applies autoupdate results.

πŸ’­ Motivation and context

Linting was broken by:

Fixed by:

βœ… Pre-approval checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • All relevant type-of-change labels have been added.
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.
  • All relevant repo and/or project documentation has been updated
    to reflect the changes in this PR.
  • All new and existing tests pass.

@felddy felddy self-assigned this Feb 20, 2023
@felddy felddy merged commit 034f78b into develop Feb 20, 2023
@felddy felddy deleted the bugfix/pre-commit branch February 20, 2023 23:51
Sallenmoore pushed a commit to Sallenmoore/foundryvtt-docker that referenced this pull request Jun 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant