Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

Use standard pre-commit config #77

Merged
merged 4 commits into from
Feb 21, 2023
Merged

Conversation

dstansby
Copy link
Member

@dstansby dstansby commented Feb 7, 2023

This applies the standard pre-commit-config.yaml file from https://github.com/SainsburyWellcomeCentre/python-cookiecutter, and fixes files as needed for this new config.

@codecov
Copy link

codecov bot commented Feb 19, 2023

Codecov Report

Base: 99.13% // Head: 99.13% // No change to project coverage 👍

Coverage data is based on head (8bd3911) compared to base (0bc0d6a).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #77   +/-   ##
=======================================
  Coverage   99.13%   99.13%           
=======================================
  Files           9        9           
  Lines         231      231           
=======================================
  Hits          229      229           
  Misses          2        2           
Impacted Files Coverage Δ
tests/tests/test_integration/test_detection.py 100.00% <ø> (ø)
tests/tests/test_unit/test_tools/test_geometry.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@dstansby dstansby marked this pull request as ready for review February 20, 2023 15:54
@dstansby dstansby requested a review from adamltyson February 20, 2023 15:54
@adamltyson adamltyson merged commit fea36d3 into brainglobe:main Feb 21, 2023
@dstansby dstansby deleted the pre-commit branch February 21, 2023 09:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants