Yaytay
Yaytay
Hi, Were you able to dig into this further? Thanks.
Interestingly I've got one project where: ```yaml - name: Checkout completely uses: actions/checkout@v3 with: fetch-depth: 0 ``` is sufficient, and another that needs: ```yaml - name: Checkout completely uses: actions/checkout@v3...
> Did one of you ask the question on the `actions/checkout` project ? I haven't.
Aye, but I don't want to change the separator, because that will confuse anyone setting these properties (as they will expect them to use dots, 'cos that's what everyone does)....
Things may have changed a lot in the last nearly three years, but looking at the props mapper now I think the issue can be resolved in the various Splitter...
Ah, this is the fundamental thing I was missing: ``` a non-zero even number of 2n contiguous backslashes before a line terminator (or elsewhere) encodes n backslashes after escape processing...
There seem to be a few areas to consider in working on an improvement and I'd like your opinion on them: 1. indexOf is going to be faster than parsing...
PR #332 as a working, minimal, attempt.
There are (at least) two problems in the first version of the PR: 1. The default behaviour should be unchanged, so the default escape char should be invalid. 2. I've...
Updated the PR. The default escape char is now '\0', which should be a safer default. Now it allows the escaping of escapes when the last escape immediately precedes the...