Skip to content
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

fix: handle new line used as separator #287

Merged
merged 20 commits into from
Apr 19, 2023

Conversation

jackton1
Copy link
Member

No description provided.

@jackton1 jackton1 linked an issue Apr 19, 2023 that may be closed by this pull request
3 tasks

> **Warning**:
>
> * Using characters like `\n`, `%`, `.` and `\r` as separators would be [URL encoded](https://www.w3schools.com/tags/ref_urlencode.asp)

Check warning

Code scanning / Markdownlint (reported by Codacy)

Expected: 1; Actual: 3

Expected: 1; Actual: 3
@@ -138,6 +138,13 @@

* Free software: [MIT license](LICENSE)


Check warning

Code scanning / Markdownlint (reported by Codacy)

Expected: 1; Actual: 2

Expected: 1; Actual: 2
entrypoint.sh Fixed Show fixed Hide fixed
entrypoint.sh Fixed Show fixed Hide fixed
@jackton1 jackton1 added the merge when passing Merge the PR automatically once all status checks have passed label Apr 19, 2023
@repo-ranger repo-ranger bot merged commit 183324b into main Apr 19, 2023
@delete-merged-branch delete-merged-branch bot deleted the fix/handle-new-line-used-as-separator branch April 19, 2023 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge when passing Merge the PR automatically once all status checks have passed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Action fails when output separator requested is newline (\n)
1 participant