-
Notifications
You must be signed in to change notification settings - Fork 6
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 python-copier-template #384
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #384 +/- ##
==========================================
+ Coverage 89.20% 89.37% +0.16%
==========================================
Files 43 43
Lines 1797 1797
==========================================
+ Hits 1603 1606 +3
+ Misses 194 191 -3 ☔ View full report in Codecov by Sentry. |
37bb6a2
to
05bc896
Compare
Start of adoption of python-copier-template (#359).
This concludes migration to python-copier-template 1.0.0
Blueapi doesn't support Python 3.8
aac160a
to
39407dd
Compare
d81c914
to
49fe911
Compare
This PR contains most of the adoption of the copier template. There will be some subsequent work
The coverage is currently failing as ruff made me fix some issues on lines not already covered. Please check the changes I have made to python code specifically. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems good, I'll check out and try running some things
docs/explanations/decisions/0004-switched-to-python-copier-template.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please could you make issues for all the next steps you've identified?
Python code looks good and I encountered no problems while playing with it
…plate.md Co-authored-by: DiamondJoseph <[email protected]>
Another issue to make:
|
10be86a
to
681ee0f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bonza
The run_engine is replaced with a mock.
47c6bf6
to
6b35b4c
Compare
#384 Moved the coverage settings in pyproject.toml from the pytest settings to the tox command. Some merge conflict kept the settings in both places, which interferes with vscode debugging. This PR removes the redundant settings.
Convert project from python3-pip-skeleton to python-copier-template version 2.0.1. This involves numerous changes most notably restructuring of documentation into a single manual. Additionally many of the pre-commit tools have been replaced with ruff. The ruff ruleset is slightly different to the previous tools so minor changes have been made to the source code. Extensions to this task for completion later: * Convert all documentation to markdown * Fix relative image handling in README * Add more detail about running blueapi tests * Enable ruff's "pyupgrade" ruleset
#384 Moved the coverage settings in pyproject.toml from the pytest settings to the tox command. Some merge conflict kept the settings in both places, which interferes with vscode debugging. This PR removes the redundant settings.
No description provided.