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

General refactor/restructure of project #303

Merged
merged 1 commit into from
Feb 9, 2023
Merged

Conversation

klaasnicolaas
Copy link
Owner

General restructure/refactor of the project.

  • Updates all sub-dependencies
  • Replace flake + all plugins + isort + pyupgrade + bandit by ruff
  • Address all issues raised by ruff
  • Simplify GitHub actions runs, to have a little fewer jobs/faster runtimes and leverage caching capabilities of other actions
  • Remove old configs no longer in use

@klaasnicolaas klaasnicolaas added major This PR causes a major version bump in the version number. refactor Improvement of existing code, not introducing new features. ci Work that improves the continue integration. labels Feb 9, 2023
@codecov
Copy link

codecov bot commented Feb 9, 2023

Codecov Report

Base: 95.80% // Head: 98.60% // Increases project coverage by +2.79% 🎉

Coverage data is based on head (d09ed4e) compared to base (67c80e0).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #303      +/-   ##
==========================================
+ Coverage   95.80%   98.60%   +2.79%     
==========================================
  Files           4        4              
  Lines         143      143              
  Branches       16       15       -1     
==========================================
+ Hits          137      141       +4     
+ Misses          3        1       -2     
+ Partials        3        1       -2     
Impacted Files Coverage Δ
p1monitor/models.py 97.14% <100.00%> (+5.36%) ⬆️
p1monitor/p1monitor.py 100.00% <100.00%> (ø)

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

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@klaasnicolaas klaasnicolaas merged commit 4c6328f into main Feb 9, 2023
@klaasnicolaas klaasnicolaas deleted the klaas-2023-002 branch May 21, 2023 10:12
@github-actions github-actions bot locked and limited conversation to collaborators Apr 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ci Work that improves the continue integration. major This PR causes a major version bump in the version number. refactor Improvement of existing code, not introducing new features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant