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

feat(build): only increase patch version on final releases #508

Conversation

paullatzelsperger
Copy link
Contributor

WHAT

This PR improves the release workflow by changing the logic where the new snapshot version is selected.

Assuming <major>.<minor>.<patch> versions, previously, we always incremented the patch version, regardless of what the input was. So 0.5.0-rc1 would have resulted in a 0.5.1-SNAPSHOT, which is incorrect.

Now, we only increase the <patch> version, if there is no -rc1 or anything:

  • 0.5.0 becomes 0.5.1-SNAPSHOT
  • 0.5.0-rc1 becomes 0.5.0-SNAPSHOT

WHY

The existing logic would have handled release candidates etc. incorrectly.

FURTHER NOTES

List other areas of code that have changed but are not necessarily linked to the main feature. This could be method signature changes, package declarations, bugs that were encountered and were fixed inline, etc.

Closes # <-- insert Issue number if one exists

@paullatzelsperger paullatzelsperger added the bug Something isn't working label Jun 20, 2023
@paullatzelsperger
Copy link
Contributor Author

\cc @jimmarino

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@paullatzelsperger paullatzelsperger merged commit 006ec6c into eclipse-tractusx:main Jun 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

3 participants