Skip to content

Sync IRIS with repo changes when switching to a remote branch #788

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

Merged
merged 4 commits into from
Jun 6, 2025

Conversation

raymond-rebbeck
Copy link
Contributor

Addresses #783

First commit re-factors the 'diff after git command' mechanism previously added in #517 so that it can be more generically applied to other Git commands.

Second commit applies it to the checkout command.

Works with Git menu dropdown and with WebUI. The diff that was previously being applied prior to the checkout being performed would fail due to the branch not yet being local at that point in time when checking out a remote branch for the first time.

…more easily applied to other commands

Use additional flags to control when and how this behaviour takes effect rather than checking for specific commands.

Will allow this to be more easily applied to other commands in the future and also to WebUI commands.
… when switching to a remote branch

Switching to a remote branch would previously not sync IRIS with the file repo changes. The diff previously performed prior to the checkout Git command would fail due to the branch not yet existing locally, with nothing being synced.
@isc-pbarton isc-pbarton self-requested a review June 6, 2025 15:05
Copy link
Collaborator

@isc-pbarton isc-pbarton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works great, thank you!

@isc-pbarton isc-pbarton merged commit 26eed61 into intersystems:main Jun 6, 2025
1 check passed
@raymond-rebbeck raymond-rebbeck deleted the remote-checkout branch June 7, 2025 03:41
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