Skip to content

[update-checkout] Fetch if the local branch/tag doesn't exist #40823

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 1 commit into from
Jan 13, 2022

Conversation

xedin
Copy link
Contributor

@xedin xedin commented Jan 12, 2022

Dependencies are usually updated to a different branch/tag name
when new versions get new released, but local checkout might not
have that fetched yet, so if initial checkout fails, let's fetch
before attempting to checkout.

@xedin xedin requested review from shahmishal and clackary January 12, 2022 19:28
@xedin xedin force-pushed the fix-update-checkout branch from 0dea8f0 to cf82cfb Compare January 12, 2022 19:28
@xedin
Copy link
Contributor Author

xedin commented Jan 12, 2022

Fresh case - experimental-string-processing switching to dev/5 which I didn't have locally.

@xedin
Copy link
Contributor Author

xedin commented Jan 12, 2022

@swift-ci please smoke test

Copy link
Member

@shahmishal shahmishal left a comment

Choose a reason for hiding this comment

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

Thanks

Dependencies are usually updated to a different branch/tag name
when new versions get new released, but local checkout might not
have that fetched yet, so if initial checkout fails, let's fetch
before attempting to checkout.
@xedin xedin force-pushed the fix-update-checkout branch from cf82cfb to 4eabfa9 Compare January 12, 2022 21:07
@xedin
Copy link
Contributor Author

xedin commented Jan 12, 2022

@swift-ci please smoke test

@xedin xedin merged commit aba448e into swiftlang:main Jan 13, 2022
@gottesmm
Copy link
Contributor

@xedin this needs a test

@AnthonyLatsis AnthonyLatsis added the update-checkout Area → utils: the `update-checkout` script label Sep 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
update-checkout Area → utils: the `update-checkout` script
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants