Skip to content

[technical debt]: updated black to 26.3.1 and reran pre-commit#1341

Open
linglp wants to merge 3 commits intodevelopfrom
update-black
Open

[technical debt]: updated black to 26.3.1 and reran pre-commit#1341
linglp wants to merge 3 commits intodevelopfrom
update-black

Conversation

@linglp
Copy link
Copy Markdown
Contributor

@linglp linglp commented Mar 20, 2026

Problem:

There's a high security risk of black 23.3.0. I updated black to the latest version of 26.3.1
see: https://github.com/Sage-Bionetworks/synapsePythonClient/security/dependabot/42
Screenshot 2026-03-20 at 10 36 19 AM

Solution:

  • updated black in pre-commit to 26.3.1
  • updated lock file to match the latest version of black
  • re-installed pre-commit and ran it to fix all files

@linglp linglp changed the title [technical debt]: update black to 26.3.1 and rerun pre-commit [technical debt]: updated black to 26.3.1 and reran pre-commit Mar 20, 2026
@thomasyu888 thomasyu888 requested a review from andrewelamb March 20, 2026 16:50
7. Delete wiki pages

"""

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is going to cause the line numbers in the tutorial to be incorrect by 1 line.

Copy link
Copy Markdown
Member

@BryanFauble BryanFauble left a comment

Choose a reason for hiding this comment

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

Thanks for the updates. I didn't see any logic changes, just formatting updates. LGTM!

@andrewelamb
Copy link
Copy Markdown
Contributor

@linglp Should we put min version of back in the setup.cfg file?

@linglp
Copy link
Copy Markdown
Contributor Author

linglp commented Mar 20, 2026

@andrewelamb

@linglp Should we put min version of back in the setup.cfg file?

I think if you just do pip install -e ".[dev]", it installs the latest version from PyPI at install time anyway.

@linglp linglp marked this pull request as ready for review March 23, 2026 16:25
@linglp linglp requested a review from a team as a code owner March 23, 2026 16:25
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.

3 participants