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

⬆️🪝 Update ruff and use automatic target-version inference #255

Merged
merged 2 commits into from
Mar 15, 2023

Conversation

burgholzer
Copy link
Member

Description

This small PR updates the configuration of the ruff pre-commit hook. Since v0.255, the target-version configuration property can be inferred directly, which allows to simplify the configuration.

Checklist:

  • The pull request only contains commits that are related to it.
  • I have added appropriate tests and documentation.
  • I have made sure that all CI jobs on GitHub pass.
  • The pull request introduces no new warnings and follows the project's style guidelines.

Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
@burgholzer burgholzer added the dependencies Pull requests that update a dependency file label Mar 15, 2023
@burgholzer burgholzer self-assigned this Mar 15, 2023
@codecov
Copy link

codecov bot commented Mar 15, 2023

Codecov Report

Merging #255 (025171c) into main (1599eac) will increase coverage by 0.1%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            main    #255     +/-   ##
=======================================
+ Coverage   96.0%   96.2%   +0.1%     
=======================================
  Files         34      34             
  Lines       1738    1738             
  Branches     214     214             
=======================================
+ Hits        1670    1672      +2     
+ Misses        68      66      -2     
Flag Coverage Δ
cpp 95.8% <ø> (+0.1%) ⬆️
python 97.3% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 2 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@burgholzer burgholzer merged commit 8f54d34 into main Mar 15, 2023
@burgholzer burgholzer deleted the update-ruff branch March 15, 2023 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant