Skip to content

Turn off merge with bare in normal merge mode#706

Merged
shijinglu merged 1 commit intotwosigma:masterfrom
shijinglu:turnOffDefaultMergeInBare
May 22, 2019
Merged

Turn off merge with bare in normal merge mode#706
shijinglu merged 1 commit intotwosigma:masterfrom
shijinglu:turnOffDefaultMergeInBare

Conversation

@shijinglu
Copy link
Collaborator

Previously we turned on merge with half-open repo by default. Now we change the default option to always open subrepos.

@novalis
Copy link
Contributor

novalis commented May 22, 2019

why?

@shijinglu
Copy link
Collaborator Author

why?

Merge has three modes, ALLOW_BARE, FORCE_OPEN and FORCE_BARE respectfully. FORCE_OPEN behaves exactly the same as the old normal merge. FORCE_BARE also behaves as expected and has been tested under different scenarios. ALLOW_BARE however this option has a bug: #707

Because the ALLOW_BARE option is only accessible from normal merge. I want to turn it off and turn it back on when the bug is fixed.

@novalis
Copy link
Contributor

novalis commented May 22, 2019 via email

@shijinglu
Copy link
Collaborator Author

Can we just fix #707?

This is the first step towards #707 's fix. I will put up it's fix later (fix is straightforward, but writing unit tests takes some time).

@shijinglu shijinglu merged commit 2a1377b into twosigma:master May 22, 2019
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

Comments