Skip to content

Actions: Fix broken escaping in PR labeller action.#2527

Merged
rdmarsh2 merged 1 commit intomasterfrom
fix-labeler-yaml-escaping
Dec 13, 2019
Merged

Actions: Fix broken escaping in PR labeller action.#2527
rdmarsh2 merged 1 commit intomasterfrom
fix-labeler-yaml-escaping

Conversation

@tausbn
Copy link
Copy Markdown
Contributor

@tausbn tausbn commented Dec 12, 2019

TL;DR: YAML escaping is complicated. I think this should fix the problems.

TL;DR: YAML escaping is complicated. I think this should fix the problems.
@tausbn
Copy link
Copy Markdown
Contributor Author

tausbn commented Dec 12, 2019

Also, important information gained: changes to Actions take effect on the PR that makes the change!

Copy link
Copy Markdown
Contributor

@rdmarsh2 rdmarsh2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rdmarsh2 rdmarsh2 merged commit 53988b4 into master Dec 13, 2019
@RasmusWL RasmusWL deleted the fix-labeler-yaml-escaping branch December 13, 2019 10:39
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.

3 participants