Skip to content

Conversation

@nelimee
Copy link
Contributor

@nelimee nelimee commented Dec 9, 2024

This PR aims at solving #13 and improving the package building / uploading process in CI.

I am unsure how to test that pushing a tag will indeed trigger the right jobs. If you have any idea, please feel free to say it. If not, I'll just experiment directly in main...

@nelimee nelimee added bug Something isn't working enhancement New feature or request labels Dec 9, 2024
@nelimee nelimee self-assigned this Dec 9, 2024
@nelimee
Copy link
Contributor Author

nelimee commented Dec 9, 2024

At this point, upload to PyPI and GitHub release seems to work fine. I have deleted both from GitHub and PyPI (because these are not supposed to be triggered from a PR), but they worked.
The only thing that may not be working correctly is the workflow triggering. As noted above, I have no idea how to test that outside of main, so I'll:

  1. rollback the changes made in this PR related to workflow triggers in order to have what I think should work,
  2. wait for this PR to be merged,
  3. iterate directly on main if the workflow is not triggered as expected.

@nelimee nelimee requested review from Gistbatch and inmzhang December 9, 2024 17:05
@github-actions
Copy link

github-actions bot commented Dec 9, 2024

Code Coverage

Package Line Rate Complexity Health
src.tqecd 94% 0
src.tqecd.match_utils 91% 0
Summary 94% (777 / 830) 0

@nelimee nelimee merged commit 30794a3 into main Dec 10, 2024
6 checks passed
@nelimee nelimee deleted the ci/manybuild branch December 10, 2024 08:14
@github-actions
Copy link

🪓 PR closed, deleted preview at https://tqec.github.io/tqecd/pull/14/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants