Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Switch to ruff as linter and formatter #413

Merged
merged 4 commits into from
Feb 5, 2024
Merged

Switch to ruff as linter and formatter #413

merged 4 commits into from
Feb 5, 2024

Conversation

MatthieuDartiailh
Copy link
Member

  • Closes # (insert issue number if relevant)
  • Executed black . && isort -c . && flake8 with no errors
  • The change is fully covered by automated unit tests
  • Documented in docs/ as appropriate
  • Added an entry to the CHANGES file

Copy link

codecov bot commented Feb 5, 2024

Codecov Report

Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (4564c89) 24.89% compared to head (e0d860e) 24.91%.
Report is 1 commits behind head on main.

Files Patch % Lines
pyvisa_py/serial.py 0.00% 6 Missing ⚠️
pyvisa_py/gpib.py 0.00% 2 Missing ⚠️
pyvisa_py/common.py 50.00% 1 Missing ⚠️
pyvisa_py/testsuite/test_sessions.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #413      +/-   ##
==========================================
+ Coverage   24.89%   24.91%   +0.02%     
==========================================
  Files          25       25              
  Lines        3515     3516       +1     
  Branches      490      489       -1     
==========================================
+ Hits          875      876       +1     
- Misses       2622     2623       +1     
+ Partials       18       17       -1     
Flag Coverage Δ
unittests 24.91% <23.07%> (+0.02%) ⬆️

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

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

@MatthieuDartiailh MatthieuDartiailh merged commit 29d013f into main Feb 5, 2024
18 of 19 checks passed
@MatthieuDartiailh MatthieuDartiailh deleted the ruff branch February 5, 2024 17:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant