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

Restructure release #242

Merged
merged 11 commits into from
Nov 19, 2024
Merged

Restructure release #242

merged 11 commits into from
Nov 19, 2024

Conversation

sroet
Copy link
Collaborator

@sroet sroet commented Nov 19, 2024

Following the debugging done in #241, I decided to restructure the release following current best practices.

The workflow now has 4 jobs each depending on the previous one to succeed, with the type of runner in []:

  1. [self-hosted] build the package and test with twine check, upload artifact
  2. [github] upload the distribution to testPyPI
  3. [self-hosted] pull the latest version from testPyPI and run the test suite
  4. [github] upload the distribution to PyPI

@sroet sroet requested a review from McHaillet November 19, 2024 15:26
Copy link

github-actions bot commented Nov 19, 2024

File Coverage Missing
All files 89%
src/pytom_tm/correlation.py 85% 97-98 130
src/pytom_tm/entry_points.py 80% 30 115-132 237-282 387-406 559-579 1092-1103
src/pytom_tm/extract.py 97% 264-269
src/pytom_tm/io.py 84% 20 40 56 72 85 104-110 127-131 138 144 181 186 233-235 305 451-459 484
src/pytom_tm/mask.py 80% 75 93 103-104
src/pytom_tm/parallel.py 89% 14-15 93 104-105 111 176
src/pytom_tm/plotting.py 16% 19-21 24-31 35-43 47-50 53-69 72-73 76-77 97-102 118-126 132-149 164-172 176-193 197-205 209-216 222 228 233 247-397
src/pytom_tm/tmjob.py 98% 371-372 388 482 610 659
src/pytom_tm/utils.py 78% 18-20
src/pytom_tm/weights.py 96% 58 313-317 324 534 539 544

Minimum allowed coverage is 88%

Generated by 🐒 cobertura-action against fa53447

@sroet sroet requested a review from FridoF November 19, 2024 15:47
@McHaillet
Copy link
Collaborator

Looks really good! I can't spot anything wrong with it :)

@sroet sroet merged commit 09f09c6 into main Nov 19, 2024
2 checks passed
@sroet sroet deleted the restructure-release branch November 20, 2024 11:35
@sroet sroet mentioned this pull request Nov 20, 2024
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.

2 participants