Skip to content

Run pyupgrade --py311-plus since python 3.11 is the min#771

Merged
atravitz merged 4 commits intomainfrom
feat/pyupgrade-py311
Mar 31, 2026
Merged

Run pyupgrade --py311-plus since python 3.11 is the min#771
atravitz merged 4 commits intomainfrom
feat/pyupgrade-py311

Conversation

@mikemhenry
Copy link
Copy Markdown
Contributor

Tips

  • Comment "pre-commit.ci autofix" to have pre-commit.ci atomically format your PR.
    Since this will create a commit, it is best to make this comment when you are finished with your work.

Checklist

  • Added a news entry

Developers certificate of origin

@mikemhenry
Copy link
Copy Markdown
Contributor Author

Need to make sure the docs build

@mikemhenry
Copy link
Copy Markdown
Contributor Author

If we do decide to merge this in, we should add the commit to .git-blame-ignore-revs

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.89%. Comparing base (266da1c) to head (4c94119).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #771   +/-   ##
=======================================
  Coverage   98.89%   98.89%           
=======================================
  Files          43       43           
  Lines        2705     2705           
=======================================
  Hits         2675     2675           
  Misses         30       30           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

@atravitz atravitz left a comment

Choose a reason for hiding this comment

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

just one comment clean up, but yes I like this, and I agree it should be added to the ignore dotfile.

Comment thread gufe/network.py Outdated
atravitz and others added 3 commits March 31, 2026 13:41
Co-authored-by: Alyssa Travitz <31974495+atravitz@users.noreply.github.com>
* include pyupgrade in ruff checks

* context manager deprecation fix
@github-actions
Copy link
Copy Markdown

No API break detected ✅

@atravitz atravitz enabled auto-merge (squash) March 31, 2026 21:22
@atravitz atravitz disabled auto-merge March 31, 2026 21:25
@atravitz atravitz merged commit bcdd946 into main Mar 31, 2026
14 checks passed
@atravitz atravitz deleted the feat/pyupgrade-py311 branch March 31, 2026 21:56
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