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

Add automatic publish to PyPI workflow with automatic release note #34

Merged
merged 6 commits into from
Oct 7, 2020

Conversation

oncleben31
Copy link
Member

The Release workflow publishes the package on PyPI. The workflow also creates a version tag in the GitHub repository, and publishes a GitHub Release using Release Drafter. The workflow is triggered on every push to the master branch.

Release steps only run if the package version was bumped. If the package version did not change, the package is instead uploaded to TestPyPI as a prerelease, and only a draft GitHub Release is created. TestPyPI is a test instance of the Python Package Index.

@oncleben31 oncleben31 requested a review from Quentame as a code owner October 6, 2020 21:25
@codecov
Copy link

codecov bot commented Oct 6, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@b9ed2b8). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             master       #34   +/-   ##
==========================================
  Coverage          ?   100.00%           
==========================================
  Files             ?        11           
  Lines             ?       387           
  Branches          ?        23           
==========================================
  Hits              ?       387           
  Misses            ?         0           
  Partials          ?         0           
Impacted Files Coverage Δ
src/meteofrance/const.py 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b9ed2b8...84e3baa. Read the comment docs.

@oncleben31 oncleben31 added the ci Continuous Integration label Oct 7, 2020
According to cjolowicz/cookiecutter-hypermodern-python#154,
there is no need to have __version__ coputed at runtime by default.
If users need it there are alternative ways to get it.
Copy link
Member

@Quentame Quentame left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, very cool CI/CD 👍 🎉

Seems nice!

Added some label release propositions.

.github/release-drafter.yml Show resolved Hide resolved
.github/release-drafter.yml Show resolved Hide resolved
.github/release-drafter.yml Show resolved Hide resolved
.github/release-drafter.yml Show resolved Hide resolved
.github/release-drafter.yml Show resolved Hide resolved
Copy link
Member

@Quentame Quentame left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ups, some English corrections

.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
Fix typo in comments

Co-authored-by: Quentame <[email protected]>
@oncleben31 oncleben31 merged commit c4ff36b into master Oct 7, 2020
@oncleben31 oncleben31 deleted the release-drafter branch October 7, 2020 15:33
@oncleben31
Copy link
Member Author

Contributes to #18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants