Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

migrated pre-commit config #771

Merged
merged 2 commits into from
Nov 11, 2024
Merged

Conversation

melonora
Copy link
Collaborator

Some of the default stage names have been deprecated for pre-commit. This PR introduces the update as suggested by pre-commit to migrate the config: pre-commit migrate-config.

Copy link

codecov bot commented Nov 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.89%. Comparing base (0aef9af) to head (9ec91cb).
Report is 69 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #771   +/-   ##
=======================================
  Coverage   91.89%   91.89%           
=======================================
  Files          45       45           
  Lines        6919     6919           
=======================================
  Hits         6358     6358           
  Misses        561      561           

@melonora melonora merged commit e27dbdd into scverse:main Nov 11, 2024
8 checks passed
@melonora melonora deleted the migrate_pre_commit branch November 11, 2024 11:07
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant