Thanks to visit codestin.com
Credit goes to github.com

Skip to content

⬆ [pre-commit.ci] pre-commit autoupdate#12115

Merged
tiangolo merged 5 commits into
masterfrom
pre-commit-ci-update-config
Sep 5, 2024
Merged

⬆ [pre-commit.ci] pre-commit autoupdate#12115
tiangolo merged 5 commits into
masterfrom
pre-commit-ci-update-config

Conversation

@pre-commit-ci

@pre-commit-ci pre-commit-ci Bot commented Sep 2, 2024

Copy link
Copy Markdown
Contributor

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.2 → v0.6.3](astral-sh/ruff-pre-commit@v0.6.2...v0.6.3)

@svlandeg svlandeg 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.

@tiangolo: when this ruff-pre-commit version is bumped, shouldn't we always simultaneously bump the ruff version in requirements-tests.txt as well?

@tiangolo

tiangolo commented Sep 4, 2024

Copy link
Copy Markdown
Member

Ideally yes, but we don't have a way to automatize it. 😔

I've been thinking if there's a way to run pre-commit with the development dependencies instead of the default custom environment for each pre-commit hook, but I haven't figured out the right workflow.

For now, up to now, I've been merging the two (dependabot and pre-commit autoupdate) independently, although it's not the idea.

@svlandeg

svlandeg commented Sep 5, 2024

Copy link
Copy Markdown
Member

Ideally yes, but we don't have a way to automatize it

Ok, no worries, for now I'll do it manually then 😇

@tiangolo tiangolo merged commit 1f64a1b into master Sep 5, 2024
@tiangolo tiangolo deleted the pre-commit-ci-update-config branch September 5, 2024 09:07
@tiangolo

tiangolo commented Sep 5, 2024

Copy link
Copy Markdown
Member

Thank you! 🙇

black-redoc pushed a commit to black-redoc/fastapi that referenced this pull request Sep 12, 2024
* ⬆ [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.2 → v0.6.3](astral-sh/ruff-pre-commit@v0.6.2...v0.6.3)

* bump ruff as well

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Sofie Van Landeghem <[email protected]>
Co-authored-by: svlandeg <[email protected]>
black-redoc pushed a commit to black-redoc/fastapi that referenced this pull request Sep 12, 2024
* ⬆ [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.2 → v0.6.3](astral-sh/ruff-pre-commit@v0.6.2...v0.6.3)

* bump ruff as well

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Sofie Van Landeghem <[email protected]>
Co-authored-by: svlandeg <[email protected]>
black-redoc pushed a commit to black-redoc/fastapi that referenced this pull request Sep 12, 2024
* ⬆ [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.2 → v0.6.3](astral-sh/ruff-pre-commit@v0.6.2...v0.6.3)

* bump ruff as well

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Sofie Van Landeghem <[email protected]>
Co-authored-by: svlandeg <[email protected]>
s-rigaud pushed a commit to s-rigaud/fastapi that referenced this pull request Jan 23, 2025
* ⬆ [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.6.2 → v0.6.3](astral-sh/ruff-pre-commit@v0.6.2...v0.6.3)

* bump ruff as well

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Sofie Van Landeghem <[email protected]>
Co-authored-by: svlandeg <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants