fix(bump): Search for version number line by line #568
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Kurt-von-Laven
force-pushed
the
line-by-line
branch
2 times, most recently
from
514d19a
to
38907bc
Compare
Avoid the complexities of multiline regex matching and inconsistencies with the case where no regex is specified. Simplify the implementation by eliminating the need to track an offset into the file. Ensure that the version number will be found even if it is to the left of the portion of the line matching the regex.
Commitizen runs bump on itself to, among other things, manage the version it uses of its own pre-commit hooks. When searching .pre-commit-config.yaml for the version number of Commitizen, we search for "Commitizen," which is in a comment to the right of the version number. Remove the lookahead assertion for "Commitizen," and instead simply consume it when matching now that version_files regexes don't need to exclusively match to the left of the version number.
Kurt-von-Laven
force-pushed
the
line-by-line
branch
from
38907bc
to
7f7a606
Compare
Codecov ReportBase: 98.43% // Head: 98.43% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #568 +/- ##
==========================================
- Coverage 98.43% 98.43% -0.01%
==========================================
Files 39 39
Lines 1602 1596 -6
==========================================
- Hits 1577 1571 -6
Misses 25 25
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
|
LGTM! @Lee-W feel free to merge |
woile
approved these changes
Aug 22, 2022
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fixes #498.
Add mypy typing for
_bump_with_regex.Avoid the complexities of multiline regex matching and inconsistencies with the case where no regex is specified. Simplify the implementation by eliminating the need to track an offset into the file. Ensure that the version number will be found even if it is to the left of the portion of the line matching the regex.
Remove lookahead assertion from
version_filesentry for.pre-commit-config.yaml. Commitizen runs bump on itself to, among other things, manage the version it uses of its own pre-commit hooks. When searching.pre-commit-config.yamlfor the version number of Commitizen, we search for "Commitizen," which is in a comment to the right of the version number. Remove the lookahead assertion for "Commitizen," and instead simply consume it when matching now thatversion_filesregexes don't need to exclusively match to the left of the version number.Checklist
./scripts/formatand./scripts/testlocally to ensure this change passes linter check and testExpected behavior
The version of Commitizen is bumped as usual in all
version_fileslisted inpyproject.toml.Steps to Test This Pull Request
cz bump.Additional context
As mentioned in #565, I am encountering some difficulties testing Commitizen locally.