Skip to content

Apply ruff/pyupgrade rule UP032 to notebooks#567

Closed
DimitriPapadopoulos wants to merge 1 commit intozarr-developers:mainfrom
DimitriPapadopoulos:ruff-0.6
Closed

Apply ruff/pyupgrade rule UP032 to notebooks#567
DimitriPapadopoulos wants to merge 1 commit intozarr-developers:mainfrom
DimitriPapadopoulos:ruff-0.6

Conversation

@DimitriPapadopoulos
Copy link
Copy Markdown
Contributor

UP032 Use f-string instead of format call

Starting with ruff 0.6, ruff also lints notebooks.

TODO:

  • Unit tests and/or doctests in docstrings
  • Tests pass locally
  • Docstrings and API docs for any new/modified user-facing classes and functions
  • Changes documented in docs/release.rst
  • Docs build locally
  • GitHub Actions CI passes
  • Test coverage to 100% (Codecov passes)

UP032 Use f-string instead of `format` call

Starting with ruff 0.6, ruff also lints notebooks.
@DimitriPapadopoulos
Copy link
Copy Markdown
Contributor Author

Duplicate of #565.

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.91%. Comparing base (2a7bf9c) to head (f1d1835).
Report is 66 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #567   +/-   ##
=======================================
  Coverage   99.91%   99.91%           
=======================================
  Files          59       59           
  Lines        2334     2334           
=======================================
  Hits         2332     2332           
  Misses          2        2           
---- 🚨 Try these New Features:

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.

1 participant