-
Notifications
You must be signed in to change notification settings - Fork 0
Move to local version of release system #5
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
Conversation
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.
Pull Request Overview
This PR migrates from using an external reusable workflow (the0mikkel/ci/.github/workflows/semver-release-standalone.yml@v1.4.1) to a local implementation of semantic-release directly within the repository's workflow file. This gives the repository more control over the release process and removes the external dependency.
Key Changes:
- Replaced reusable workflow call with inline semantic-release implementation
- Added semantic-release dependency installation and configuration steps
- Implemented default configuration file creation with conventional commits support
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Pull Request Overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| npx semantic-release 2>&1 | tee semantic-release.log | ||
| status=${PIPESTATUS[0]} | ||
| exit $status | ||
| - name: Archive release log |
Copilot
AI
Nov 16, 2025
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 semantic-release log artifact upload should be conditional to handle cases where the Release step fails before creating the log file. Add if: always() to ensure the artifact upload runs even if previous steps fail:
- name: Archive release log
if: always()
uses: actions/upload-artifact@v5| - name: Archive release log | |
| - name: Archive release log | |
| if: always() |
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.
Pull Request Overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.