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 some of the Python configuration #280

Merged
merged 5 commits into from
Aug 22, 2023
Merged

Conversation

burgholzer
Copy link
Member

This PR adds a proper pytest and coverage configuration and also some further ruff checks and corresponding auto-fixes.

Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
@burgholzer burgholzer added dependencies Pull requests that update a dependency file python Pull requests that update Python code labels Aug 11, 2023
@burgholzer burgholzer self-assigned this Aug 11, 2023
@codecov
Copy link

codecov bot commented Aug 11, 2023

Codecov Report

Merging #280 (a8a1893) into main (4416a6f) will increase coverage by 0.1%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            main    #280     +/-   ##
=======================================
+ Coverage   94.9%   95.1%   +0.1%     
=======================================
  Files         20      20             
  Lines       1951    1950      -1     
  Branches     348     348             
=======================================
+ Hits        1853    1855      +2     
+ Misses        98      95      -3     

see 1 file with indirect coverage changes

@hillmich hillmich disabled auto-merge August 22, 2023 11:48
@hillmich hillmich merged commit c6f95af into main Aug 22, 2023
19 checks passed
@hillmich hillmich deleted the python-config-update branch August 22, 2023 13:38
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 python Pull requests that update Python code
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants