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
Build sdist in CI #388
Build sdist in CI #388
Conversation
Codecov Report
@@ Coverage Diff @@
## master #388 +/- ##
=======================================
Coverage 94.57% 94.57%
=======================================
Files 16 16
Lines 792 792
=======================================
Hits 749 749
Misses 43 43 |
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.
Overall look good to me! Finally you found a way to release anonlink yay!
- '*' | ||
tags: | ||
include: | ||
- v* |
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.
Why do we need a tag starting with v? Is this pipeline only for release?
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.
the release pipeline is triggered by the build pipeline. In the past, we didn't execute the build pipeline when a tag is set, thus, the release pipeline was never automatically triggered.
I added a stage to the CI that builds the source distribution and publishes it to the 'drop' artifact.
And modified the publish to test feed section to include the source dist.
Currently, the release pipeline only publishes the wheel files. We also need to publish the source distribution though.
Additionally, the source distribution was incomplete, as the manifest file was not up to date.
And to round it all off, I also updated the ubuntu images in the CI script, as 16.04 is quite old now. (fixes #385)