Re: [PATCH 1/4] stash: convert apply to builtin
- From
Joel Teichroeb <joel@teichroeb.net>
- Date
- Mar 25, 2018, 16:51 UTC
- Message-ID
- <CA+CzEk9QpmHK_TSBwQfEedNqrcVSBp3xY7bdv1YA_KxePiFeXw@mail.gmail.com>
- In-Reply-To
- <CAP8UFD3Qxt2YMqTtHwU8n7EDvD66QjGSywRQoxJDnncv7=2BUg@mail.gmail.com>
On Sun, Mar 25, 2018 at 1:09 AM, Christian Couder <christian.couder@gmail.com> wrote:
> It seems to me that the apply_stash() shell function is also used in > pop_stash() and in apply_to_branch(). Can the new helper be used there > too instead of apply_stash()? And then could apply_stash() be remove?
I wasn't really sure if I should remove code from the .sh file as it seems in the past the old .sh files have been kept around as examples. Has that been done for previous conversions?