Visitar URL original
Update pinned requirements 📍 by plasmapy-requirements-bot[bot] · Pull Request #2909 · PlasmaPy/PlasmaPy · GitHub
Skip to content

Update pinned requirements 📍 - #2909

Merged
namurphy merged 9 commits into
mainfrom
create-pull-request/patch
Nov 26, 2024
Merged

Update pinned requirements 📍#2909
namurphy merged 9 commits into
mainfrom
create-pull-request/patch

Conversation

@plasmapy-requirements-bot

@plasmapy-requirements-bot plasmapy-requirements-bot Bot commented Nov 15, 2024

Copy link
Copy Markdown
Contributor

This pull request (PR) regenerates the requirements files in ci_requirements/ that are used for running tests, building documentation, and performing other continuous integration (CI) checks. Pinning requirements files reduces the probability that a CI check will spontaneously start failing in a PR due to a breaking change in a dependency. This PR lets us find unexpected breaking changes in newly released dependencies before they get used in CI. 🛡

If all checks pass ✅, please merge this PR. If any checks fail due to a breaking change in a dependency 🚨, please fix the problem and get the checks to pass again.

Note

These changes were generated by running nox -s requirements in the top-level directory of the repository, using the requirements session defined in noxfile.py. This Nox session uses uv to regenerate requirements files. The workflow is defined in .github/workflows/update-pinned-reqs.yml. For more information, see ci_requirements/README.md.

Important

When it is necessary to put a temporary upper limit on the allowed versions of a dependency in pyproject.toml, please create an issue that this upper limit should be removed.

@plasmapy-requirements-bot plasmapy-requirements-bot Bot added no changelog entry needed For PRs that should not have a changelog entry requirements Related to updating requirements labels Nov 15, 2024
@github-actions

github-actions Bot commented Nov 15, 2024

Copy link
Copy Markdown

Thank you for submitting this pull request (PR)! ✨ PlasmaPy depends on contributors like you, so we deeply appreciate it!

The bottom of this page shows several checks that are run for every PR. ✅ Don't worry if something broke! We break stuff all the time. 😅 Click on "Details" to learn why a check didn't pass. Please also feel free to ask for help. We do that all the time as well! 🌸 We invite you to our weekly community meeting (2 pm ET on most Tuesdays) and office hours (3 pm ET on most Thursdays).

After this PR is marked as ready for review, someone should stop by to provide a code review and offer suggestions soon. 👥 If we don't provide a review within a few days, please feel free to send us a reminder. 🎗

PlasmaPy's contributor guide has information on:

Tip

📚 For a documentation preview, click on Details next to docs/readthedocs.org:plasmapy. For cryptic documentation errors, see the documentation troubleshooting guide.

Tip

🧹 Automatically fix most pre-commit.ci failures by commenting pre-commit.ci autofix below. For other failures, please see the pre-commit troubleshooting guide.

We thank you once again! 🌌

@codecov

codecov Bot commented Nov 15, 2024 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.49%. Comparing base (9983e00) to head (b493dc6).
Report is 24 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #2909      +/-   ##
==========================================
- Coverage   95.28%   94.49%   -0.79%     
==========================================
  Files         108      108              
  Lines        9631     9631              
  Branches     1451     1451              
==========================================
- Hits         9177     9101      -76     
- Misses        273      343      +70     
- Partials      181      187       +6

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

namurphy
namurphy reviewed Nov 15, 2024
View reviewed changes
Comment thread ci_requirements/all-3.13-macos.txt Resolved
namurphy
namurphy reviewed Nov 15, 2024
View reviewed changes
Comment thread ci_requirements/all-3.13-macos.txt Resolved
namurphy
namurphy reviewed Nov 15, 2024
View reviewed changes
Comment thread ci_requirements/all-3.13-macos.txt Resolved
@github-actions github-actions Bot added the packaging Related to packaging or distribution label Nov 15, 2024
This was referenced Nov 15, 2024
Closed
Merged
namurphy
namurphy reviewed Nov 15, 2024
View reviewed changes
Comment thread ci_requirements/docs-3.13-linux.txt Resolved
namurphy
namurphy reviewed Nov 15, 2024
View reviewed changes
Comment thread ci_requirements/docs-3.13-linux.txt Resolved
@namurphy namurphy changed the title Update pinned requirements 📍 Test if recent dependency updates are causing the Read the Docs failure Nov 15, 2024
@namurphy

namurphy commented Nov 15, 2024

Copy link
Copy Markdown
Member

My conclusion from this pull request is that the recent updates to dependencies are probably not causing the RTD build errors.

@plasmapy-requirements-bot plasmapy-requirements-bot Bot changed the title Test if recent dependency updates are causing the Read the Docs failure Update pinned requirements 📍 Nov 18, 2024
@namurphy

namurphy commented Nov 25, 2024

Copy link
Copy Markdown
Member

This is stalled since we need to figure out the intersphinx issue in #2915 and the RTD build error in #2908. I expect to get to this next week after submitting a proposal.

@namurphy
namurphy requested review from a team, pheuer and rocco8773 as code owners November 26, 2024 22:07
@github-actions github-actions Bot added testing plasmapy.diagnostics Related to the plasmapy.diagnostics subpackage labels Nov 26, 2024
@github-actions github-actions Bot added plasmapy.utils Related to the plasmapy.utils subpackage linters Code linters and autoformatters maintenance General updates to package infrastructure python Pull requests that update Python code labels Nov 26, 2024
@namurphy

namurphy commented Nov 26, 2024

Copy link
Copy Markdown
Member

I'm adding a temporary upper limit on Astropy until we fix the problem in #2883. We'll need to resolve this before our next release.

@namurphy

namurphy commented Nov 26, 2024

Copy link
Copy Markdown
Member

I also added an upper limit on xarray and raised #2922 because there was a new failure in test_grids.py associated with better Quantity support in xarray.

@namurphy

namurphy commented Nov 26, 2024

Copy link
Copy Markdown
Member

This PR is a reminder that packages that are left alone will slowly become obsolete due to changes in the upstream environment. 😅

It's also a good thing that I have the 😅 emoji on speed-dial. 😅

namurphy
namurphy approved these changes Nov 26, 2024
View reviewed changes

@namurphy namurphy left a comment

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.

Took a while but this is finally ready to merge! 😅

@namurphy
namurphy enabled auto-merge (squash) November 26, 2024 22:55
@namurphy
namurphy merged commit fa66c1f into main Nov 26, 2024
@github-actions github-actions Bot removed the no changelog entry needed For PRs that should not have a changelog entry label Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@namurphy namurphy namurphy approved these changes

Assignees

@namurphy namurphy

Labels

linters Code linters and autoformatters maintenance General updates to package infrastructure packaging Related to packaging or distribution plasmapy.diagnostics Related to the plasmapy.diagnostics subpackage plasmapy.utils Related to the plasmapy.utils subpackage python Pull requests that update Python code requirements Related to updating requirements testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@namurphy