Skip to content

fix: issue #55 format tests with black - #161

Merged
arobson-ods merged 3 commits into
developfrom
ar/fix-issue-55
Aug 5, 2026
Merged

fix: issue #55 format tests with black#161
arobson-ods merged 3 commits into
developfrom
ar/fix-issue-55

Conversation

@arobson-ods

Copy link
Copy Markdown
Contributor

Fixes #55 - format tests with black

Also tightens the black include regex to only match files ending in .py. Previously the unescaped period matched any character, so it could incorrectly match non-.py files (e.g. a file ending in "...npy").

@arobson-ods
arobson-ods requested a review from simon-20 August 5, 2026 09:11

@simon-20 simon-20 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple of comments in line. Only other thing is: could we split up the commit so that the change to the config is in one commit and the actual running of the formatting job is another commit (with chore:).

Comment thread tests/integration/test_dataset_upload_to_azure.py Outdated
Comment thread tests/integration/test_dataset_upload_to_azure.py Outdated
@simon-20
simon-20 self-requested a review August 5, 2026 13:15
@arobson-ods
arobson-ods merged commit 5bd2135 into develop Aug 5, 2026
1 check passed
@arobson-ods
arobson-ods deleted the ar/fix-issue-55 branch August 5, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Format tests with black

2 participants