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

fix: transpile static js to IE 10 #2063

Merged
merged 1 commit into from
Nov 28, 2019

Conversation

JLHwung
Copy link
Contributor

@JLHwung JLHwung commented Nov 27, 2019

Motivation

Transpile static JavaScript files to IE 10 support so that they does not bail on IE 10.

This Babel REPL is broken on IE 11 due to the client JavaScript requires ES6.

Have you read the [Contributing Guidelines on pull requests] (https://github.com/facebook/docusaurus/blob/master/CONTRIBUTING.md#pull-requests)?

Yep

Test Plan

Manually testing the generated website.

Related PRs

(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Nov 27, 2019
@@ -5,57 +5,64 @@
* LICENSE file in the root directory of this source tree.
*/

/* eslint-disable prefer-arrow-callback */
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is required to prevent ESLint rule auto fix like prefer-shorthand.

@docusaurus-bot
Copy link
Contributor

Deploy preview for docusaurus-2 ready!

Built with commit 89b7667

https://deploy-preview-2063--docusaurus-2.netlify.com

@docusaurus-bot
Copy link
Contributor

Deploy preview for docusaurus-preview ready!

Built with commit 89b7667

https://deploy-preview-2063--docusaurus-preview.netlify.com

@endiliey endiliey added the pr: bug fix This PR fixes a bug in a past release. label Nov 28, 2019
@endiliey endiliey changed the title transpile static js to IE 10 fix: transpile static js to IE 10 Nov 28, 2019
Copy link
Contributor

@endiliey endiliey left a comment

Choose a reason for hiding this comment

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

Thank you

Copy link
Contributor

@yangshun yangshun left a comment

Choose a reason for hiding this comment

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

Thank you, it's tricky to have to remember to not write ES6 for these scripts :|

@yangshun yangshun merged commit 34e942e into facebook:master Nov 28, 2019
@JLHwung
Copy link
Contributor Author

JLHwung commented Nov 29, 2019

@yangshun Yeah. If the team does plan to change these files often, we can use eslint-plugin-node/es-syntax and specify things like node <= 0.10 so it should bail out on any ES6 syntaxes.

If we are not going to change these files, then I would say just leave it as-is.

@JLHwung JLHwung deleted the add-ie10-support-1x-static branch November 30, 2019 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA pr: bug fix This PR fixes a bug in a past release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants