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

✨ adopt ruff formatter and update ruff config #388

Merged
merged 1 commit into from
Oct 30, 2023
Merged

Conversation

burgholzer
Copy link
Member

Description

This PR adopts the new ruff formatter as a replacement for black and, in turn, also updates the ruff 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.

@burgholzer burgholzer self-assigned this Oct 30, 2023
@burgholzer burgholzer added dependencies Pull requests that update a dependency file python Anything related to Python code continuous integration Anything related to the CI setup code quality Anything related to code quality and code style. labels Oct 30, 2023
@burgholzer burgholzer enabled auto-merge (squash) October 30, 2023 21:25
@codecov
Copy link

codecov bot commented Oct 30, 2023

Codecov Report

Merging #388 (1df16ad) into main (0708e68) will decrease coverage by 0.1%.
The diff coverage is 100.0%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main    #388     +/-   ##
=======================================
- Coverage   92.3%   92.3%   -0.1%     
=======================================
  Files         44      44             
  Lines       4162    4160      -2     
  Branches     701     701             
=======================================
- Hits        3842    3840      -2     
  Misses       320     320             
Flag Coverage Δ *Carryforward flag
cpp 91.8% <ø> (ø) Carriedforward from 0708e68
python 95.8% <100.0%> (-0.1%) ⬇️

*This pull request uses carry forward flags. Click here to find out more.

Files Coverage Δ
src/mqt/qmap/subarchitectures.py 96.3% <100.0%> (-0.1%) ⬇️

@burgholzer burgholzer merged commit b6aad60 into main Oct 30, 2023
26 checks passed
@burgholzer burgholzer deleted the ruff-update branch October 30, 2023 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code quality Anything related to code quality and code style. continuous integration Anything related to the CI setup dependencies Pull requests that update a dependency file python Anything related to Python code
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant