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 version scheme, by avoiding version jump on pre-releases #674

Merged
merged 1 commit into from
Mar 14, 2023

Conversation

y0urself
Copy link
Member

@y0urself y0urself commented Mar 10, 2023

What

Fixing next_minor_version() and next_major_version()

Why

  • Currently when having a zero pre version, e.g. 0.5.0a1 it will be released to 0.6.0 when doing a minor release, same for major releases, e.g. 1.0.0a1 will get 2.0.0 instead of the correct 1.0.0.

References

None.

Checklist

  • Tests

@y0urself y0urself force-pushed the y0urself/correct-versioning branch from 83d72a3 to a7beb54 Compare March 10, 2023 10:14
@y0urself y0urself marked this pull request as ready for review March 10, 2023 10:15
@y0urself y0urself requested a review from a team as a code owner March 10, 2023 10:15
@codecov
Copy link

codecov bot commented Mar 10, 2023

Codecov Report

Merging #674 (0be1f6b) into main (7a2cebf) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #674   +/-   ##
=======================================
  Coverage   89.45%   89.46%           
=======================================
  Files          79       79           
  Lines        4950     4954    +4     
  Branches      661      663    +2     
=======================================
+ Hits         4428     4432    +4     
  Misses        368      368           
  Partials      154      154           
Impacted Files Coverage Δ
pontos/version/_calculator.py 100.00% <100.00%> (ø)
pontos/version/schemes/_semantic.py 90.67% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@y0urself y0urself marked this pull request as draft March 10, 2023 10:19
@y0urself
Copy link
Member Author

On hold for major changes in the Version module of pontos regarding SemVer.

@y0urself y0urself force-pushed the y0urself/correct-versioning branch from a7beb54 to be915c6 Compare March 14, 2023 13:01
@y0urself y0urself marked this pull request as ready for review March 14, 2023 13:01
@y0urself y0urself enabled auto-merge (squash) March 14, 2023 13:01
@y0urself y0urself force-pushed the y0urself/correct-versioning branch from be915c6 to 0be1f6b Compare March 14, 2023 13:28
@y0urself y0urself merged commit 0b4793a into main Mar 14, 2023
@y0urself y0urself deleted the y0urself/correct-versioning branch March 14, 2023 13:36
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