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(config): Add configuration for javascript version. #1939

Conversation

Moumi
Copy link
Contributor

@Moumi Moumi commented Feb 28, 2016

Add configuration for a javascript version tag. This tag is only applied to Firefox (now).

Closes #1719

@@ -59,9 +60,21 @@ var getXUACompatibleUrl = function (url) {
return value
}

function applyJavascriptVersionTag (req) {
Copy link
Member

Choose a reason for hiding this comment

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

Better name would be isFirefox or hasJsVersionTags

sjelin and others added 4 commits March 2, 2016 17:41
…, intelligently merge the list of preprocessors, deduping and trying to preserve the order

This could be a breaking change, some users might rely on the old order, or on some preprocessors being run multiple times
Change information on how to use node.js on Windows
The javascript version tag would only be applied to the Firefox browser.
@Moumi Moumi force-pushed the feature-javascript-version-tag-firefox branch from 91102e0 to 78320db Compare March 2, 2016 16:45
@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

sjelin and others added 4 commits March 2, 2016 17:47
…, intelligently merge the list of preprocessors, deduping and trying to preserve the order

This could be a breaking change, some users might rely on the old order, or on some preprocessors being run multiple times
Change information on how to use node.js on Windows
feat(config): Add configuration for adding a javascript version.
@Moumi
Copy link
Contributor Author

Moumi commented Mar 2, 2016

This PR is getting a bit messy. Closing it to create a new one.

@Moumi Moumi closed this Mar 2, 2016
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.

5 participants