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

fix: 🐛 Fix incorrect usage of semver tool #26

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

DRovara
Copy link
Contributor

@DRovara DRovara commented Aug 19, 2024

The semver tool that is downloaded by this workflow does not provide the option -i.

Instead, the equivalent command is semver bump patch <version>

Before this pull request, updating the version using this workflow will fail, as -i is not recognised. This pull request should fix the issue.

@burgholzer burgholzer added continuous integration Anything related to the CI setup fix Fix for something that is not working github_actions Pull requests that update Github_actions code labels Aug 19, 2024
Copy link
Member

@burgholzer burgholzer left a comment

Choose a reason for hiding this comment

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

Many thanks for catching this! 🚀
Don't know how I missed that before 🙃

I'll create a new release right after merging!

@burgholzer burgholzer merged commit 821a111 into cda-tum:main Aug 19, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
continuous integration Anything related to the CI setup fix Fix for something that is not working github_actions Pull requests that update Github_actions code
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants