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

rustdoc: Fix string escaping in implementors js files #40451

Merged
merged 1 commit into from
Mar 13, 2017

Conversation

ollie27
Copy link
Member

@ollie27 ollie27 commented Mar 12, 2017

The generated HTML can contain quotes so we need to make sure they are
escaped before inserting into the js files.

The generates HTML can contain quotes so we need to make sure they are
escaped before inserting into the js files.
@rust-highfive
Copy link
Collaborator

r? @steveklabnik

(rust_highfive has picked a reviewer for you, use r? to override)

@GuillaumeGomez
Copy link
Member

Thanks!

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Mar 13, 2017

📌 Commit 19e0505 has been approved by GuillaumeGomez

@bors
Copy link
Contributor

bors commented Mar 13, 2017

⌛ Testing commit 19e0505 with merge a5483a7...

bors added a commit that referenced this pull request Mar 13, 2017
rustdoc: Fix string escaping in implementors js files

The generated HTML can contain quotes so we need to make sure they are
escaped before inserting into the js files.
@bors
Copy link
Contributor

bors commented Mar 13, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: GuillaumeGomez
Pushing a5483a7 to master...

@bors bors merged commit 19e0505 into rust-lang:master Mar 13, 2017
@ollie27 ollie27 deleted the rustdoc_impls_js_escape branch March 14, 2017 11:30
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.

5 participants