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

automate a common build process to check for "use strict" safe code #2251

Merged
merged 2 commits into from
Nov 5, 2019

Conversation

joshgoebel
Copy link
Member

@joshgoebel joshgoebel commented Nov 1, 2019

  • Adds a test to the CI pipeline that builds a "typical" consolidated
    Highlight.js package from the full NPM source and then checks that
    that build actually executes without errors

@joshgoebel joshgoebel changed the title automate a common build process to check for "use strict" safe code WIP: automate a common build process to check for "use strict" safe code Nov 1, 2019
@joshgoebel joshgoebel changed the title WIP: automate a common build process to check for "use strict" safe code automate a common build process to check for "use strict" safe code Nov 1, 2019
- Adds a test to the CI pipeline that builds a "typical" consolidated
  Highlight.js package from the full NPM source and then checks that
  that build actually executes without errors

Related to highlightjs#2247.
@joshgoebel
Copy link
Member Author

joshgoebel commented Nov 1, 2019

Related: #2247

@egor-rogov If you have thoughts on naming or a better approach here I'm all ears. We could build the executable in tmp or something instead?

@joshgoebel joshgoebel added bug package/build Issues relating to npm or packaging labels Nov 1, 2019
Copy link
Collaborator

@egor-rogov egor-rogov left a comment

Choose a reason for hiding this comment

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

Looks fine to me (from what little I know about ES).

@joshgoebel joshgoebel merged commit e946496 into highlightjs:master Nov 5, 2019
@joshgoebel joshgoebel deleted the test_use_strict_safe branch December 6, 2019 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug package/build Issues relating to npm or packaging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants