update deduce_base to include error handingling #83

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.

Sign up for GitHub

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

Jump to bottom

Open

AdityaKhowalGithub wants to merge 1 commit into modular:main from AdityaKhowalGithub:fixing_crash_on_origin

AdityaKhowalGithub commented Apr 28, 2025 •

edited

Loading

Summary

Improves error handling in the deduce_base function to prevent crashes when remote branches are missing or merge-base fails.

Changes

Added check for remote branch existence
Added graceful error handling for merge-base failures
Added clear error messages to help users resolve issues

Error Examples

Missing branch: "ERROR: Remote branch 'origin/main' does not exist"
Merge-base failure: "ERROR: Failed to find a merge base"

Testing

Verified the tool exits gracefully with appropriate messages when remote branch is missing or merge-base fails.


          update deduce_base to include error handingling

0d11d32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet