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

When updating a version ignore the current version in case of erros #804

Merged

Conversation

bjoernricks
Copy link
Contributor

What

When updating a version ignore the current version in case of erros

Why

During a version update the project's current version is parsed. This may result in VersionErrors if for example the versioning scheme is changed and the new versioning scheme doesn't recognize the current version format (PEP440 based 22.4.1.dev1 can't be parsed as semver).

The current version is just parsed during the update to check if it is a real update or if the same version is set. Therefore ignore the VersionErrors during the version update and just set the new version.

@bjoernricks bjoernricks requested a review from a team as a code owner July 3, 2023 06:27
@bjoernricks bjoernricks enabled auto-merge (rebase) July 3, 2023 06:28
@github-actions
Copy link

github-actions bot commented Jul 3, 2023

Conventional Commits Report

Type Number
Bug Fixes 1

🚀 Conventional commits found.

…rors

During a version update the project's current version is parsed. This
may result in VersionErrors if for example the versioning scheme is
changed and the new versioning scheme doesn't recognize the current
version format (PEP440 based 22.4.1.dev1 can't be parsed as semver).

The current version is just parsed during the update to check if it is a
real update or if the same version is set. Therefore ignore the
VersionErrors during the version update and just set the new version.
@bjoernricks bjoernricks force-pushed the ignore-current-version-error-during-version-update branch from 47d16f8 to bfa662d Compare July 3, 2023 07:02
@codecov
Copy link

codecov bot commented Jul 3, 2023

Codecov Report

Merging #804 (bfa662d) into main (f7a75b7) will increase coverage by 0.06%.
The diff coverage is 84.21%.

@@            Coverage Diff             @@
##             main     #804      +/-   ##
==========================================
+ Coverage   89.67%   89.74%   +0.06%     
==========================================
  Files          82       82              
  Lines        5406     5421      +15     
  Branches      762      762              
==========================================
+ Hits         4848     4865      +17     
+ Misses        390      389       -1     
+ Partials      168      167       -1     
Impacted Files Coverage Δ
pontos/version/commands/_cmake.py 94.00% <66.66%> (-1.88%) ⬇️
pontos/version/commands/_java.py 82.05% <66.66%> (-1.95%) ⬇️
pontos/version/commands/_cargo.py 86.66% <75.00%> (-3.81%) ⬇️
pontos/version/commands/_go.py 100.00% <100.00%> (ø)
pontos/version/commands/_javascript.py 89.28% <100.00%> (+1.63%) ⬆️
pontos/version/commands/_python.py 93.40% <100.00%> (+8.17%) ⬆️

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

@bjoernricks bjoernricks merged commit 1845f1a into main Jul 3, 2023
13 of 14 checks passed
@bjoernricks bjoernricks deleted the ignore-current-version-error-during-version-update branch July 3, 2023 07:17
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