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

Update CITATION.cff and release through CI/CD #84

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

lukaspie
Copy link
Collaborator

@lukaspie lukaspie commented Sep 25, 2024

With this PR, we could update the CITATION.cff file, make a new PR, and then (conditionally on that PR being approved), make a new release, all using GitHub actions.

One needs two new workflows:

  1. update_citation.yaml: This action is manually dispatched, you need to pass the version tag and whether or not you want to create a new pre-release. It checks whether the supplied version is a correct semver version and then creates a new PR where the version in CITATION.cff is updated.
  2. release-on-citation-update.yaml: This one is triggered as soon as that PR is merged into main. It creates a new release using the version that we initially set up.

The whole release process would then be 1) go to Actions, manually trigger the first action (see image), and 2) approve and merge the created PR.
image

Currently, this chain does not work though because GitHub actions is not allowed to create PRs. This has to be allowed on the organization level in GitHub.
image

@lukaspie lukaspie marked this pull request as draft September 25, 2024 14:19
@coveralls
Copy link

coveralls commented Sep 25, 2024

Pull Request Test Coverage Report for Build 11034947315

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 10815454559: 0.0%
Covered Lines: 48
Relevant Lines: 48

💛 - Coveralls

@lukaspie lukaspie force-pushed the release-through-cicd branch from b0fc6c6 to 2eabe61 Compare September 25, 2024 14:35
@lukaspie lukaspie force-pushed the release-through-cicd branch from 2eabe61 to 5f108eb Compare September 25, 2024 14:41
@lukaspie lukaspie force-pushed the release-through-cicd branch from c754997 to 8a3f51c Compare October 2, 2024 09:16
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