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

chore: update pre-commit hooks #2215

Merged
merged 3 commits into from
Feb 15, 2023
Merged

chore: update pre-commit hooks #2215

merged 3 commits into from
Feb 15, 2023

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Feb 7, 2023

@pre-commit-ci pre-commit-ci bot temporarily deployed to docs-preview February 7, 2023 02:11 Inactive
@@ -16,7 +16,7 @@
overload,
)

ShapeItem: TypeAlias = "SupportsInt | None"
Copy link
Collaborator

Choose a reason for hiding this comment

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

@henryiii do you know which ruff rule is wiping out these quotes?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, update Ruff to v0.0.243, it was noted in astral-sh/ruff#2613 and fixed in astral-sh/ruff#2614 and released.

@codecov
Copy link

codecov bot commented Feb 7, 2023

Codecov Report

Merging #2215 (ecd8c24) into main (6b49960) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
Impacted Files Coverage Δ
src/awkward/_connect/jax/reducers.py 82.26% <ø> (ø)
src/awkward/_connect/jax/trees.py 91.89% <ø> (ø)
src/awkward/_connect/numexpr.py 90.00% <ø> (ø)
src/awkward/_connect/rdataframe/from_rdataframe.py 0.00% <ø> (ø)
src/awkward/_connect/rdataframe/to_rdataframe.py 0.00% <ø> (ø)
src/awkward/_do.py 84.39% <ø> (ø)
src/awkward/_util.py 74.42% <ø> (ø)
src/awkward/contents/listarray.py 88.84% <ø> (ø)
src/awkward/contents/numpyarray.py 91.34% <ø> (ø)
src/awkward/contents/recordarray.py 84.97% <ø> (ø)
... and 16 more

@pre-commit-ci pre-commit-ci bot temporarily deployed to docs-preview February 7, 2023 23:05 Inactive
@agoose77 agoose77 temporarily deployed to docs-preview February 8, 2023 11:48 — with GitHub Actions Inactive
@agoose77
Copy link
Collaborator

@jpivarski are you happy to upgrade black here? (I am)

@pre-commit-ci pre-commit-ci bot temporarily deployed to docs-preview February 13, 2023 10:40 Inactive
updates:
- [github.com/psf/black: 22.12.0 → 23.1.0](psf/black@22.12.0...23.1.0)
- [github.com/charliermarsh/ruff-pre-commit: v0.0.237 → v0.0.246](astral-sh/ruff-pre-commit@v0.0.237...v0.0.246)
@pre-commit-ci pre-commit-ci bot temporarily deployed to docs-preview February 14, 2023 00:39 Inactive
@agoose77 agoose77 temporarily deployed to docs-preview February 15, 2023 15:04 — with GitHub Actions Inactive
@agoose77 agoose77 merged commit cc37b99 into main Feb 15, 2023
@agoose77 agoose77 deleted the pre-commit-ci-update-config branch February 15, 2023 16:01
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