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 #531

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

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

wants to merge 1 commit into from

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Jun 17, 2024 •
edited
Loading

User description

updates:

PR Type

dependencies, configuration changes

Description

  • Updated pre-commit-hooks from v4.5.0 to v4.6.0.
  • Updated ruff-pre-commit from v0.4.4 to v0.4.9.

Changes walkthrough 📝

Relevant files
Dependencies
.pre-commit-config.yaml
Update pre-commit hooks and ruff-pre-commit versions         

.pre-commit-config.yaml

  • Updated pre-commit-hooks from v4.5.0 to v4.6.0.
  • Updated ruff-pre-commit from v0.4.4 to v0.4.9.
    		•  +2/-2     

    💡 PR-Agent usage:
    Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    @github-actions github-actions bot added dependencies Pull requests that update a dependency file configuration changes Review effort [1-5]: 1 labels Jun 17, 2024
    @github-actions GitHub Actions
    Copy link
    Contributor

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review [1-5] 1
    🧪 Relevant tests No
    🔒 Security concerns No
    ⚡ Key issues to review None

    @github-actions GitHub Actions
    Copy link
    Contributor

    PR Code Suggestions ✨

    CategorySuggestion                                                                                                                                    Score
    Enhancement
    Update the hook revisions to the latest available versions

    Ensure that the updated revision numbers for the pre-commit hooks are the latest available
    versions. This is crucial to leverage the most recent features and fixes. If v4.6.0 for
    pre-commit-hooks and v0.4.9 for ruff-pre-commit are not the latest, update them
    accordingly.

    .pre-commit-config.yaml [5-12]

    -rev: v4.6.0
-rev: v0.4.9
+rev: [latest_version]
+rev: [latest_version]
    Suggestion importance[1-10]: 7

    Why: Ensuring that the pre-commit hooks use the latest versions is good practice for leveraging new features and fixes. However, the suggestion does not specify what the latest versions are, making it less actionable.

    		7

    @pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 8ee63eb to 1b19c72 Compare June 24, 2024 22:47
    @pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from 0937a5d to 4ce23aa Compare July 8, 2024 22:25
    @pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 4ce23aa to b37ef35 Compare July 15, 2024 22:53
    @pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from 654e669 to 4ec18de Compare July 29, 2024 22:53
    @pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from 3dc15ff to 306388b Compare August 12, 2024 22:53
    @codecov Codecov
    Copy link

    codecov bot commented Aug 12, 2024 •
    edited
    Loading

    Codecov Report

    All modified and coverable lines are covered by tests ✅

    Project coverage is 93.42%. Comparing base (d0cf601) to head (dd051cd).
    Report is 25 commits behind head on master.

    Additional details and impacted files 
    @@            Coverage Diff             @@
##           master     #531      +/-   ##
==========================================
+ Coverage   93.39%   93.42%   +0.02%     
==========================================
  Files          31       31              
  Lines        3286     3299      +13     
==========================================
+ Hits         3069     3082      +13     
  Misses        217      217

    ☔ View full report in Codecov by Sentry.
    📢 Have feedback on the report? Share it here.

    @pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 306388b to dd051cd Compare August 19, 2024 22:55
    @pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from 5ecef33 to 14c51bd Compare September 2, 2024 22:52
    @pre-commit-ci
    [pre-commit.ci] pre-commit autoupdate
    c490e68 
    updates:
- [github.com/pre-commit/pre-commit-hooks: v4.5.0 → v5.0.0](pre-commit/pre-commit-hooks@v4.5.0...v5.0.0)
- [github.com/astral-sh/ruff-pre-commit: v0.4.4 → v0.6.9](astral-sh/ruff-pre-commit@v0.4.4...v0.6.9)
    @pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from d0a9281 to c490e68 Compare October 8, 2024 01:13
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Reviewers
    No reviews
    Assignees
    No one assigned
    Labels
    configuration changes dependencies Pull requests that update a dependency file Review effort [1-5]: 1
    Projects
    None yet
    Milestone
    No milestone
    Development

    Successfully merging this pull request may close these issues.
    0 participants