search

ubermag / discretisedfield

Python package for the analysis and visualisation of finite-difference fields.
http://ubermag.github.io
BSD 3-Clause "New" or "Revised" License
18 stars 13 forks source link

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

Open pre-commit-ci[bot] opened 4 months ago

pre-commit-ci[bot] commented 4 months ago

User description

updates:

PR Type

dependencies, configuration changes

Description

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[bot] commented 4 months ago

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review [1-5] 1
    🧪 Relevant tests No
    🔒 Security concerns No
    ⚡ Key issues to review None
    github-actions[bot] commented 4 months ago

    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]](https://github.com/ubermag/discretisedfield/pull/531/files#diff-63a9c44a44acf85fea213a857769990937107cf072831e1a26808cfde9d096b9R5-R12) ```diff -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
    codecov[bot] commented 2 months ago

    Codecov Report

    All modified and coverable lines are covered by tests :white_check_mark:

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

    Additional details and impacted files ```diff @@ 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 ```

    :umbrella: View full report in Codecov by Sentry.
    :loudspeaker: Have feedback on the report? Share it here.