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 python-copier-template #384

Merged
merged 56 commits into from
Mar 15, 2024
Merged

Adopt python-copier-template #384

merged 56 commits into from
Mar 15, 2024

Conversation

joeshannon
Copy link
Contributor

No description provided.

Copy link

codecov bot commented Mar 6, 2024

Codecov Report

Attention: Patch coverage is 93.75000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 89.37%. Comparing base (4b5514e) to head (6b35b4c).

Files Patch % Lines
src/blueapi/utils/serialization.py 0.00% 1 Missing ⚠️
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.
📢 Have feedback on the report? Share it here.

@joeshannon joeshannon force-pushed the skeleton-to-copier branch 10 times, most recently from 37bb6a2 to 05bc896 Compare March 8, 2024 18:19
@joeshannon joeshannon changed the title Skeleton to copier Adopt python-copier-template Mar 14, 2024
@joeshannon joeshannon marked this pull request as ready for review March 14, 2024 17:19
@joeshannon
Copy link
Contributor Author

This PR contains most of the adoption of the copier template. There will be some subsequent work

  • Convert all docs to markdown (at the moment they are stll rst apart from the new files introduced by copier)
  • Fix relative image handling in README (not really realated to copier adoption)
  • Add some specific info lost from the run-tests page
  • Enable rufff's UP issues and fix them

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.

Copy link
Collaborator

@DiamondJoseph DiamondJoseph left a 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/conf.py Show resolved Hide resolved
Copy link
Collaborator

@callumforrester callumforrester left a 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

@joeshannon
Copy link
Contributor Author

Another issue to make:

/scratch/30day_tmp/delete05mar/copier/blueapi/venv/lib/python3.10/site-packages/coverage/inorout.py:519: CoverageWarning: Module blueapi was previously imported, but not measured (module-not-measured)
  self.warn(msg, slug="module-not-measured")

Copy link
Collaborator

@DiamondJoseph DiamondJoseph left a 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.
@joeshannon joeshannon merged commit 9356536 into main Mar 15, 2024
25 checks passed
@joeshannon joeshannon deleted the skeleton-to-copier branch March 15, 2024 13:50
joeshannon pushed a commit that referenced this pull request Apr 26, 2024
#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.
ZohebShaikh pushed a commit that referenced this pull request May 7, 2024
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
ZohebShaikh pushed a commit that referenced this pull request May 7, 2024
#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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants