Sitelet https://web.archive.org/web/20220523061821/https://github.com/localstack/localstack/pull/5778
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

Upgrade black #5778

Merged
merged 2 commits into from Apr 1, 2022
Merged

Upgrade black #5778

merged 2 commits into from Apr 1, 2022

Conversation

silv-io
Copy link
Member

@silv-io silv-io commented Mar 31, 2022

Upgrade black to fix issue with click addressed here: psf/black#2964

@silv-io silv-io requested a review from alexrashed Mar 31, 2022
@silv-io silv-io had a problem deploying to localstack-ext-tests Mar 31, 2022 Failure
@silv-io silv-io had a problem deploying to localstack-ext-tests Mar 31, 2022 Failure
Copy link
Member

@alexrashed alexrashed left a comment

Great catch, thanks for the fix! Nice to have black on the latest major version again as well. :)

@github-actions
Copy link

@github-actions github-actions bot commented Mar 31, 2022 •

LocalStack integration with Pro

       3 files  ±0         3 suites  ±0   50m 35s ⏱️ - 5m 47s
   941 tests ±0     903 ✔️ ±0  38 💤 ±0  0 ❌ ±0 
1 143 runs  ±0  1 082 ✔️ ±0  61 💤 ±0  0 ❌ ±0 

Results for commit 81565e2. ± Comparison against base commit 81487c8.

♻️ This comment has been updated with latest results.

@silv-io silv-io temporarily deployed to localstack-ext-tests Apr 1, 2022 Inactive
@coveralls
Copy link

@coveralls coveralls commented Apr 1, 2022

Coverage Status

Coverage decreased (-0.04%) to 91.464% when pulling 81565e2 on upgrade-black into 81487c8 on master.

@silv-io silv-io merged commit 1dcdd37 into master Apr 1, 2022
14 checks passed
@viren-nadkarni viren-nadkarni deleted the upgrade-black branch Apr 3, 2022
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.

None yet

3 participants