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

Example currently broken in latest/published versions #52

Merged
merged 1 commit into from
Oct 4, 2017

Conversation

ericduran
Copy link
Contributor

@ericduran ericduran commented Oct 4, 2017

The broken example causes the highlight script to completely screw it up.

So it currently looks like this:


Preview | Diff

@ericduran
Copy link
Contributor Author

Oh, it looks like the official repos are locked from outside contributions.

But this is good to know. I'll keep any changes in the official repos as non-substantive so as to not cause any problems ;-)

@ericduran
Copy link
Contributor Author

Also question, since this is broken is both latest and published, Should I also open a PR with the same commit on L2 or V2, not quite sure which is which?

@plehegar
Copy link
Member

plehegar commented Oct 4, 2017

Marked as non substantive for IPR from ash-nazg.

@plehegar
Copy link
Member

plehegar commented Oct 4, 2017

LGTM. No need to a separate for the published version. we'll push an update once this gets merged.

@igrigorik igrigorik merged commit eb78a11 into w3c:gh-pages Oct 4, 2017
@igrigorik
Copy link
Member

👍

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