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

Feat(global): add hightlighter and beautifier as deps #756

Merged
merged 1 commit into from May 13, 2016

Conversation

marcoscaceres
Copy link
Member

  • adds to package dependencies: highlight.js and jsbeautifier
  • adds default options for beautifier (beautify-options.js)
  • tells builder to include highlighter and beautifier
  • Sets appropriate paths in profile-w3c-common.js

  * adds to package dependencies: highlight.js and jsbeautifier
  * adds default options for beautifier (beautify-options.js)
  * tells builder to include highlighter and beautifier
  * Sets appropriate paths in profile-w3c-common.js
@marcoscaceres
Copy link
Member Author

This is Part 1 of some incoming pull requests....

@marcoscaceres
Copy link
Member Author

Once these land, I can send the rest.

@dontcallmedom
Copy link
Member

@marcoscaceres I've looked at the changes and they look OK, but I was not sure what observable changes in the output of ReSpec I should be looking for — can you say a bit more?

@marcoscaceres
Copy link
Member Author

@dontcallmedom nothing will change yet. This is just laying down the infrastructure. All the cool stuff if coming next :)

It's all in this branch:
https://github.com/w3c/respec/commits/745_hightlighjs

@marcoscaceres
Copy link
Member Author

@dontcallmedom, I have to send it in pieces, as they all build on each other and it's a lot of code that's about to come in. I also want to make sure tests keep passing, etc.

@dontcallmedom dontcallmedom merged commit 52121ad into develop May 13, 2016
@marcoscaceres marcoscaceres deleted the highlight_beautify branch May 13, 2016 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants