Skip to content

Conversation

krittick
Copy link
Contributor

Summary

Removes pre-commit requirement, uses Github actions for workflows instead.

Fixes broken pre-commit library behavior that their author doesn't want to fix.

Checklist

  • If code changes were made then they have been tested.
    • I have updated the documentation to reflect the changes.
  • If type: ignore comments were used, a comment is also left explaining why
  • This PR fixes an issue.
  • This PR adds something new (e.g. new method or parameters).
  • This PR is a breaking change (e.g. methods or parameters removed/renamed)
  • This PR is not a code change (e.g. documentation, README, typehinting, examples, ...)

Thanks to @VincentRPS for the guidance on using a better method for this.

@krittick krittick added priority: high High Priority bypass-review Bypasses reviews for urgent pull requests involving critical code or documentation fixes. Merge with squash labels Feb 15, 2022
@krittick krittick self-assigned this Feb 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bypass-review Bypasses reviews for urgent pull requests involving critical code or documentation fixes. priority: high High Priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant