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

Pre-release workflow #1989

Merged
merged 19 commits into from
May 2, 2022
Merged

Conversation

epassaro
Copy link
Member

@epassaro epassaro commented Apr 28, 2022

Description

The pre-release workflow prepares TARDIS for a new release by just creating a single pull request to this repository:

  • Runs scheduled (weekly) or manually
  • Updates .zenodo.json metadata (no more need to writing directly to the protected master branch ❌💀)
  • Updates CHANGELOG.md (and it works 😆 ) move to a post-release pipeline
  • Creates .lock files according to TEP - 013
  • Opens a PR to tardis-sn/tardis with all these changes
  • A clear workflow for future file additions (like CITATION.cff, updated README.rst, etc.) move to a post-release pipeline

Then, if all checks pass, one of us should merge the pull request and an upcoming workflow should make the actual release.

Motivation and context

Make a more secure, robust release cycle.

How has this been tested?

  • Testing pipeline.
  • Other.

Tested on my fork and works as expected. In practice this workflow just creates a pull request, really safe to merge.

Examples

Type of change

  • Bug fix.
  • New feature.
  • Breaking change.
  • None of the above.

Checklist

  • My change requires a change to the documentation.
    • I have updated the documentation accordingly.
    • (optional) I have built the documentation on my fork following the instructions.
  • I have assigned and requested two reviewers for this pull request.

@codecov
Copy link

codecov bot commented Apr 28, 2022

Codecov Report

Merging #1989 (cc53945) into master (ae19f26) will not change coverage.
The diff coverage is n/a.

❗ Current head cc53945 differs from pull request most recent head 2a89ab2. Consider uploading reports for the commit 2a89ab2 to get more accurate results

@@           Coverage Diff           @@
##           master    #1989   +/-   ##
=======================================
  Coverage   59.99%   59.99%           
=======================================
  Files          70       70           
  Lines        8111     8111           
=======================================
  Hits         4866     4866           
  Misses       3245     3245           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@epassaro epassaro self-assigned this Apr 29, 2022
@epassaro epassaro merged commit d1dffde into tardis-sn:master May 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants