Skip to content
This repository has been archived by the owner on May 8, 2024. It is now read-only.

Adjust clone depth for PRs #249

Merged
merged 1 commit into from
Sep 23, 2022
Merged

Adjust clone depth for PRs #249

merged 1 commit into from
Sep 23, 2022

Conversation

fkorotkov
Copy link
Contributor

Since the recent change in #247

Otherwise it fails with CIRRUS_CLONE_DEPTH is set to 1 which we don't recommend but people do it.

@fkorotkov
Adjust clone depth for PRs
6a045de 
Since the recent change in #247
@fkorotkov fkorotkov requested a review from edigaryev September 23, 2022 19:11
@fkorotkov fkorotkov enabled auto-merge (squash) September 23, 2022 19:12
@fkorotkov fkorotkov disabled auto-merge September 23, 2022 19:17
@fkorotkov fkorotkov merged commit 8ff4962 into master Sep 23, 2022
@fkorotkov fkorotkov deleted the fixed-pr-clone-depth branch September 23, 2022 19:17
fkorotkov added a commit that referenced this pull request Sep 24, 2022
@fkorotkov
Fall back to checking out HEAD
0e577f1 
In case merge ref is not available. These edge cases were cauhgt by integration tests in Cirrus CI:

1. Cloning a PR with conflicts.
2. Cloning a closed PR.

Related to #247 and #249
@fkorotkov fkorotkov mentioned this pull request Sep 24, 2022
Merged
fkorotkov added a commit that referenced this pull request Sep 25, 2022
@fkorotkov
Fall back to checking out HEAD (#250)
7db20cf 
* Fall back to checking out HEAD

In case merge ref is not available. These edge cases were cauhgt by integration tests in Cirrus CI:

1. Cloning a PR with conflicts.
2. Cloning a closed PR.

Related to #247 and #249

* Use FetchContext
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@edigaryev edigaryev Awaiting requested review from edigaryev

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@fkorotkov