Skip to content
This repository has been archived by the owner on Jan 8, 2019. It is now read-only.

Bug 1449848 - Bugfix for JSON substitutions (+ prettify) #92

Merged
merged 2 commits into from
Apr 3, 2018

Conversation

djmitche
Copy link
Contributor

@djmitche djmitche commented Apr 3, 2018

No description provided.

@djmitche djmitche self-assigned this Apr 3, 2018
@djmitche djmitche requested a review from imbstack April 3, 2018 17:44
@djmitche djmitche changed the title Bugfix json subs Bug 1449848 - Bugfix for JSON substitutions (+ prettify) Apr 3, 2018
Copy link
Contributor

@imbstack imbstack left a comment

Choose a reason for hiding this comment

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

Tests are failing but otherwise makes sense.

@djmitche
Copy link
Contributor Author

djmitche commented Apr 3, 2018

Hm, now this will escape the JSON too:

\[
  1,
  2,
  3
\]

@djmitche
Copy link
Contributor Author

djmitche commented Apr 3, 2018

Maybe @eliperelman is right in taskcluster/taskcluster-rfcs#117, and we should just render these as preformatted text, not markdown.

@djmitche
Copy link
Contributor Author

djmitche commented Apr 3, 2018

I'm going to land this anyway, as it's ugly but not vulnerable.

@djmitche djmitche merged commit f2733c2 into taskcluster:master Apr 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants