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

Improve error reporting for git errors #203

Merged
merged 3 commits into from
Jun 25, 2024

Conversation

coordt
Copy link
Member

@coordt coordt commented Jun 25, 2024

This will making debugging issues like #202 easier.

coordt added 2 commits June 25, 2024 11:07
A test has been added to the bumpversion library to ensure that no commit and tag is made if there is an error modification. Specifically, the test checks the "do_bump" function and asserts that "mock_commit_and_tag" and "mock_update_config_file" are not called under these conditions.
The error message for failures in the SCM command execution has been enhanced. Now it displays not only the command's return code but also the standard output and error, improving the debugging process.
@coordt coordt linked an issue Jun 25, 2024 that may be closed by this pull request
Copy link

Version hint: minor
Current version: 0.23.0
New version (when merged): 0.24.0

Comment ID: Display the version hint-auto-generated

Copy link

github-actions bot commented Jun 25, 2024

PR Preview Action v1.4.7
🚀 Deployed preview to https://callowayproject.github.io/bump-my-version/pr-preview/pr-203/
on branch gh-pages at 2024-06-25 16:12 UTC

The code in test_bump.py file has been modified to include a test for VersionNotFoundError exception. This ensures that the implementation properly handles cases where a specified version could not be found.
@coordt coordt merged commit 5506ace into master Jun 25, 2024
13 checks passed
@coordt coordt deleted the 202-calledprocesserror-git-commit-f-tmp branch June 25, 2024 16:13
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.

CalledProcessError git commit -F /tmp
1 participant