Skip to content

Document the stash.useBuiltin escape hatch #193

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

Closed
wants to merge 1 commit into from

Conversation

dscho
Copy link
Member

@dscho dscho commented May 14, 2019

Shamelessly copy-edited from Ævar's d8d0a54 (rebase doc: document rebase.useBuiltin, 2018-11-14)

:-D

Cc: Ævar Arnfjörð Bjarmason [email protected]

The stash.useBuiltin variable introduced in 90a4627 ("stash:
optionally use the scripted version again", 2019-02-25) was turned on by
default, but had no documentation.

Let's document it so that users who run into any stability issues with
the C rewrite know there's an escape hatch, and spell out that the
user should please report the bug when they have to turn off the
built-in stash.

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho
Copy link
Member Author

dscho commented May 14, 2019

/submit

@gitgitgadget
Copy link

gitgitgadget bot commented May 14, 2019

Submitted as [email protected]

@gitgitgadget
Copy link

gitgitgadget bot commented May 15, 2019

This branch is now known as js/stash-in-c-use-builtin-doc.

@gitgitgadget
Copy link

gitgitgadget bot commented May 15, 2019

This patch series was integrated into pu via git@9042b3e.

@gitgitgadget gitgitgadget bot added the pu label May 15, 2019
@gitgitgadget
Copy link

gitgitgadget bot commented May 16, 2019

This patch series was integrated into pu via git@d3dadaa.

@gitgitgadget
Copy link

gitgitgadget bot commented May 16, 2019

This patch series was integrated into next via git@26c8bfc.

@gitgitgadget gitgitgadget bot added the next label May 16, 2019
@gitgitgadget
Copy link

gitgitgadget bot commented May 19, 2019

This patch series was integrated into pu via git@4ecaede.

@gitgitgadget
Copy link

gitgitgadget bot commented May 19, 2019

This patch series was integrated into next via git@4ecaede.

@gitgitgadget
Copy link

gitgitgadget bot commented May 19, 2019

This patch series was integrated into master via git@4ecaede.

@gitgitgadget gitgitgadget bot added the master label May 19, 2019
@gitgitgadget gitgitgadget bot closed this May 19, 2019
@gitgitgadget
Copy link

gitgitgadget bot commented May 19, 2019

Closed via 4ecaede.

@dscho dscho deleted the builtin-stash-knob-docs branch May 20, 2019 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant