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.

Sign up for GitHub

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

Jump to bottom

[pre-commit.ci] pre-commit autoupdate #14781

Merged
merged 3 commits into from
May 8, 2024
Merged

[pre-commit.ci] pre-commit autoupdate #14781

merged 3 commits into from
May 8, 2024

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Apr 15, 2024 •
edited
Loading

@pre-commit-ci pre-commit-ci bot requested review from a team as code owners April 15, 2024 17:04
@Gauravtalreja1 Gauravtalreja1 added CherryPick PR needs CherryPick to previous branches 6.12.z Introduced in or relating directly to Satellite 6.12 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 labels Apr 15, 2024
@Gauravtalreja1
Copy link
Collaborator

pre-commit.ci autofix

@ogajduse
Copy link
Member

We might want to change our config a bit before we merge this one. See https://docs.astral.sh/ruff/configuration/.

They have modified the configuration path for some options i.e.

[tool.ruff]
# Allow fix for all enabled rules (when `--fix`) is provided.
fixable = ["ALL"]
unfixable = []

should be changed to

[tool.ruff.lint]
# Allow fix for all enabled rules (when `--fix`) is provided.
fixable = ["ALL"]
unfixable = []

@ogajduse
Copy link
Member 

warning: The top-level linter settings are deprecated in favour of their counterparts in the `lint` section. Please update the following options in `pyproject.toml`:
  - 'fixable' -> 'lint.fixable'
  - 'ignore' -> 'lint.ignore'
  - 'select' -> 'lint.select'
  - 'flake8-quotes' -> 'lint.flake8-quotes'
  - 'flake8-pytest-style' -> 'lint.flake8-pytest-style'
  - 'isort' -> 'lint.isort'
  - 'mccabe' -> 'lint.mccabe'

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 968e041 to 0ab69f8 Compare April 22, 2024 16:59
@Gauravtalreja1
Copy link
Collaborator

@ogajduse Could you open a separate PR updating the ruff configuration (or add a commit to same PR), then rebase this PR once it's merged?

@Gauravtalreja1 Gauravtalreja1 self-requested a review April 24, 2024 19:20
@ogajduse ogajduse mentioned this pull request Apr 25, 2024
Merged
@ogajduse
Copy link
Member

@ogajduse Could you open a separate PR updating the ruff configuration (or add a commit to same PR), then rebase this PR once it's merged?

see #14885

@ogajduse ogajduse force-pushed the pre-commit-ci-update-config branch from 0ab69f8 to 3f4b0e1 Compare April 25, 2024 15:41
@ogajduse
Copy link
Member

Will catch up on this one tomorrow

@ogajduse ogajduse mentioned this pull request Apr 26, 2024
Merged
@Gauravtalreja1
Copy link
Collaborator

pre-commit.ci autofix

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 1cdd324 to 5b2d444 Compare April 29, 2024 17:05
@Gauravtalreja1
Copy link
Collaborator

pre-commit.ci autofix

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 5b2d444 to ad3d4d2 Compare May 6, 2024 17:01
pre-commit-ci bot and others added 2 commits May 8, 2024 14:41
@pre-commit-ci @Gauravtalreja1
[pre-commit.ci] pre-commit autoupdate
c885c61 
updates:
- [github.com/pre-commit/pre-commit-hooks: v4.4.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.4.0...v4.6.0)
- [github.com/astral-sh/ruff-pre-commit: v0.4.1 → v0.4.3](astral-sh/ruff-pre-commit@v0.4.1...v0.4.3)
- [github.com/gitleaks/gitleaks: v8.18.0 → v8.18.2](gitleaks/gitleaks@v8.18.0...v8.18.2)
@ogajduse @Gauravtalreja1
Fix UP031 - Use format specifiers instead of percent format
346f1ee
@Gauravtalreja1 Gauravtalreja1 merged commit a71fa30 into master May 8, 2024
8 checks passed
@Gauravtalreja1 Gauravtalreja1 deleted the pre-commit-ci-update-config branch May 8, 2024 15:27
github-actions bot pushed a commit that referenced this pull request May 8, 2024
@pre-commit-ci @web-flow
[pre-commit.ci] pre-commit autoupdate (#14781)
fd0f98e 
* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.4.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.4.0...v4.6.0)
- [github.com/astral-sh/ruff-pre-commit: v0.4.1 → v0.4.3](astral-sh/ruff-pre-commit@v0.4.1...v0.4.3)
- [github.com/gitleaks/gitleaks: v8.18.0 → v8.18.2](gitleaks/gitleaks@v8.18.0...v8.18.2)

* Fix UP031 - Use format specifiers instead of percent format

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ondřej Gajdušek <[email protected]>
Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit a71fa30)
github-actions bot pushed a commit that referenced this pull request May 8, 2024
@pre-commit-ci @web-flow
[pre-commit.ci] pre-commit autoupdate (#14781)
8fa4fae 
* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.4.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.4.0...v4.6.0)
- [github.com/astral-sh/ruff-pre-commit: v0.4.1 → v0.4.3](astral-sh/ruff-pre-commit@v0.4.1...v0.4.3)
- [github.com/gitleaks/gitleaks: v8.18.0 → v8.18.2](gitleaks/gitleaks@v8.18.0...v8.18.2)

* Fix UP031 - Use format specifiers instead of percent format

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ondřej Gajdušek <[email protected]>
Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit a71fa30)
github-actions bot pushed a commit that referenced this pull request May 8, 2024
@pre-commit-ci @web-flow
[pre-commit.ci] pre-commit autoupdate (#14781)
21dfe63 
* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.4.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.4.0...v4.6.0)
- [github.com/astral-sh/ruff-pre-commit: v0.4.1 → v0.4.3](astral-sh/ruff-pre-commit@v0.4.1...v0.4.3)
- [github.com/gitleaks/gitleaks: v8.18.0 → v8.18.2](gitleaks/gitleaks@v8.18.0...v8.18.2)

* Fix UP031 - Use format specifiers instead of percent format

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ondřej Gajdušek <[email protected]>
Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit a71fa30)
This was referenced May 8, 2024
Merged
Merged
Merged
Closed
Gauravtalreja1 pushed a commit that referenced this pull request May 8, 2024
@Satellite-QE @pre-commit-ci
[6.14.z] [pre-commit.ci] pre-commit autoupdate (#15001)
90d4418 
[pre-commit.ci] pre-commit autoupdate (#14781)

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.4.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.4.0...v4.6.0)
- [github.com/astral-sh/ruff-pre-commit: v0.4.1 → v0.4.3](astral-sh/ruff-pre-commit@v0.4.1...v0.4.3)
- [github.com/gitleaks/gitleaks: v8.18.0 → v8.18.2](gitleaks/gitleaks@v8.18.0...v8.18.2)

* Fix UP031 - Use format specifiers instead of percent format

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ondřej Gajdušek <[email protected]>
Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit a71fa30)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Gauravtalreja1 pushed a commit that referenced this pull request May 8, 2024
@Satellite-QE @pre-commit-ci
[6.15.z] [pre-commit.ci] pre-commit autoupdate (#15000)
a7c3123 
[pre-commit.ci] pre-commit autoupdate (#14781)

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.4.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.4.0...v4.6.0)
- [github.com/astral-sh/ruff-pre-commit: v0.4.1 → v0.4.3](astral-sh/ruff-pre-commit@v0.4.1...v0.4.3)
- [github.com/gitleaks/gitleaks: v8.18.0 → v8.18.2](gitleaks/gitleaks@v8.18.0...v8.18.2)

* Fix UP031 - Use format specifiers instead of percent format

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ondřej Gajdušek <[email protected]>
Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit a71fa30)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
ogajduse pushed a commit to ogajduse/robottelo that referenced this pull request May 10, 2024
@pre-commit-ci @ogajduse
[pre-commit.ci] pre-commit autoupdate (SatelliteQE#14781)
4f5ec2e 
* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.4.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.4.0...v4.6.0)
- [github.com/astral-sh/ruff-pre-commit: v0.4.1 → v0.4.3](astral-sh/ruff-pre-commit@v0.4.1...v0.4.3)
- [github.com/gitleaks/gitleaks: v8.18.0 → v8.18.2](gitleaks/gitleaks@v8.18.0...v8.18.2)

* Fix UP031 - Use format specifiers instead of percent format

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ondřej Gajdušek <[email protected]>
Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit a71fa30)
@ogajduse ogajduse mentioned this pull request May 10, 2024
Merged
Gauravtalreja1 pushed a commit that referenced this pull request May 10, 2024
@ogajduse @pre-commit-ci
[6.12.z] [pre-commit.ci] pre-commit autoupdate (#14781) (#15030)
1e8d168 
[pre-commit.ci] pre-commit autoupdate (#14781)

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.4.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.4.0...v4.6.0)
- [github.com/astral-sh/ruff-pre-commit: v0.4.1 → v0.4.3](astral-sh/ruff-pre-commit@v0.4.1...v0.4.3)
- [github.com/gitleaks/gitleaks: v8.18.0 → v8.18.2](gitleaks/gitleaks@v8.18.0...v8.18.2)

* Fix UP031 - Use format specifiers instead of percent format

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ondřej Gajdušek <[email protected]>
Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit a71fa30)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Gauravtalreja1 pushed a commit that referenced this pull request May 10, 2024
@Satellite-QE @pre-commit-ci @ogajduse
[6.13.z] [pre-commit.ci] pre-commit autoupdate (#14999)
6958b6d 
* [pre-commit.ci] pre-commit autoupdate (#14781)

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.4.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.4.0...v4.6.0)
- [github.com/astral-sh/ruff-pre-commit: v0.4.1 → v0.4.3](astral-sh/ruff-pre-commit@v0.4.1...v0.4.3)
- [github.com/gitleaks/gitleaks: v8.18.0 → v8.18.2](gitleaks/gitleaks@v8.18.0...v8.18.2)

* Fix UP031 - Use format specifiers instead of percent format

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ondřej Gajdušek <[email protected]>
Co-authored-by: Gaurav Talreja <[email protected]>
(cherry picked from commit a71fa30)

* Fix UP031 Use format specifiers instead of percent format

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ondřej Gajdušek <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ogajduse ogajduse ogajduse approved these changes

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

Assignees
No one assigned
Labels
6.12.z Introduced in or relating directly to Satellite 6.12 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 CherryPick PR needs CherryPick to previous branches
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Gauravtalreja1 @ogajduse