-
Notifications
You must be signed in to change notification settings - Fork 22
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
fix: Proper release workflow #727
fix: Proper release workflow #727
Conversation
- Instead of the separate workflow update-docs-for-release.yml, the release workflow now directly performs what the separate workflow did. - Accordingly, the separate workflow has been removed.
refactor: Release workflow
run: sed -i "s|${{ steps.main-changelog-update.outputs.previous-irs-version }}|${{ inputs.irs-version }}|" ${{ env.IRS_APPLICATION_PATH }} | ||
|
||
- name: Update irs-api.yaml | ||
uses: mikefarah/[email protected] |
Check notice
Code scanning / KICS
Unpinned Actions Full Length Commit SHA Note
…refactor/release-workflow # Conflicts: # .github/workflows/release.yaml
chore: updated docs and Helm chart for release
chore: updated docs and Helm chart for release
run: sed -i "s|${{ steps.main-changelog-update.outputs.previous-irs-version }}|${{ inputs.irs-version }}|" ${{ env.IRS_APPLICATION_PATH }} | ||
|
||
- name: Update irs-api.yaml | ||
uses: mikefarah/[email protected] |
Check notice
Code scanning / KICS
Unpinned Actions Full Length Commit SHA
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.
LGTM
5ea1f3b
into
eclipse-tractusx:chore/improved-release-workflow
* refactor(release): improve release workflow
Description
This PR changes the workflow
release.yml
in that the workflow now performs what was previously done via a separate workflowupdate-docs-for-release.yml
which updated the docs upon release.Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review: