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

pre-commit ruff 0.8.0 update #7595

Merged
merged 2 commits into from
Nov 28, 2024

Conversation

juanitorduz
Copy link
Contributor

@juanitorduz juanitorduz commented Nov 28, 2024

Fixes the errors in #7587 . Some changes due to ruff 0.8.0 (see https://github.com/astral-sh/ruff/releases/tag/0.8.0)

Most of the changes were generated by the pre-commit hook (see fa7d0b1). Just a couple of manual fixes.


📚 Documentation preview 📚: https://pymc--7595.org.readthedocs.build/en/7595/

@juanitorduz juanitorduz added pre-commit no releasenotes Skipped in automatic release notes generation labels Nov 28, 2024
@juanitorduz juanitorduz requested review from jessegrabowski and ricardoV94 and removed request for jessegrabowski November 28, 2024 15:54
@juanitorduz
Copy link
Contributor Author

@ricardoV94 squash + merge 🟢 ?

Copy link

codecov bot commented Nov 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.84%. Comparing base (fe0e0d7) to head (7b8a025).
Report is 3 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #7595   +/-   ##
=======================================
  Coverage   92.84%   92.84%           
=======================================
  Files         106      106           
  Lines       17686    17686           
=======================================
  Hits        16421    16421           
  Misses       1265     1265           
Files with missing lines Coverage Δ
pymc/backends/__init__.py 91.66% <100.00%> (ø)
pymc/data.py 89.22% <ø> (ø)
pymc/distributions/__init__.py 100.00% <ø> (ø)
pymc/distributions/continuous.py 98.25% <ø> (ø)
pymc/distributions/discrete.py 99.40% <ø> (ø)
pymc/distributions/distribution.py 91.89% <ø> (ø)
pymc/distributions/multivariate.py 93.58% <ø> (ø)
pymc/distributions/shape_utils.py 90.90% <ø> (ø)
pymc/distributions/timeseries.py 94.77% <ø> (ø)
pymc/distributions/transforms.py 98.47% <ø> (ø)
... and 20 more

@ricardoV94
Copy link
Member

@ricardoV94 squash + merge 🟢 ?

sure

@juanitorduz juanitorduz merged commit d2761a3 into pymc-devs:main Nov 28, 2024
23 checks passed
@juanitorduz juanitorduz deleted the fix-pre-commit-ruff8 branch November 28, 2024 17:02
@juanitorduz
Copy link
Contributor Author

😄 👌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no releasenotes Skipped in automatic release notes generation pre-commit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants