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

chore: pin black version #2747

Merged
merged 3 commits into from
Jan 30, 2024

Conversation

dbluhm
Copy link
Contributor

@dbluhm dbluhm commented Jan 30, 2024

Though relatively infrequent, the result of formatting with black can change from version to version. This PR pins the version of black used in the github action to a specific version. This version should be kept in sync with the version selected in the pyproject.toml to avoid situations where new versions of black cause PR checks to fail because formatting changed between versions.

This PR also bumps the currently selected version of black to 24.1.1, which is the most recent black version that was run in recent PRs. This should provide the best continuity until the next time we choose to update black.

Should correct the issues seen in #2737 and #2746 (black was passing locally).

@dbluhm dbluhm requested a review from swcurran January 30, 2024 00:57
@dbluhm dbluhm force-pushed the chore/pin-black-version branch from 10a849c to d7b69c3 Compare January 30, 2024 00:57
Signed-off-by: Daniel Bluhm <[email protected]>
Signed-off-by: Daniel Bluhm <[email protected]>
@dbluhm dbluhm force-pushed the chore/pin-black-version branch from 75625c3 to 1b92a0b Compare January 30, 2024 17:17
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@ianco ianco enabled auto-merge January 30, 2024 17:21
@ianco ianco merged commit e6a1e7a into openwallet-foundation:main Jan 30, 2024
8 checks passed
@dbluhm dbluhm deleted the chore/pin-black-version branch January 30, 2024 18:48
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.

2 participants