Skip to content

ENHANCEMENT | BBL-528 added alt default branch and logic to find out whether main or master is the default branch#34

Merged
juanmatias merged 2 commits intomasterfrom
BBL-528-bug-review-Forked-repos-sync-pipeline
Oct 17, 2022
Merged

ENHANCEMENT | BBL-528 added alt default branch and logic to find out whether main or master is the default branch#34
juanmatias merged 2 commits intomasterfrom
BBL-528-bug-review-Forked-repos-sync-pipeline

Conversation

@juanmatias
Copy link
Contributor

@juanmatias juanmatias commented Oct 12, 2022

what

  • nowadays there are a lot of git repositories set with main as default branch (instead of master)
  • this change is about determining what's the default branch to work on it
  • e.g. when checking out the default branch use the right name (i.e. master or main)

why

  • the makefiles fail when trying to checkout master on a non-master repo

N/A

@juanmatias juanmatias added enhancement New feature or request patch labels Oct 12, 2022
@juanmatias juanmatias requested a review from a team as a code owner October 12, 2022 13:26
@juanmatias juanmatias mentioned this pull request Oct 12, 2022
Merged
@exequielrafaela exequielrafaela requested a review from a team October 12, 2022 20:40
@exequielrafaela exequielrafaela added this to the 2022 Q4 milestone Oct 12, 2022
@juanmatias juanmatias merged commit 32018a5 into master Oct 17, 2022
@juanmatias juanmatias deleted the BBL-528-bug-review-Forked-repos-sync-pipeline branch October 17, 2022 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@exequielrafaela exequielrafaela exequielrafaela approved these changes

Assignees

No one assigned

Labels

enhancement New feature or request patch

Projects

None yet

Milestone

2022 Q4

Development

Successfully merging this pull request may close these issues.

2 participants

@juanmatias @exequielrafaela