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 adding javascript version. #8

Closed
wants to merge 1 commit into from

Conversation

Moumi
Copy link

@Moumi Moumi commented Mar 8, 2016

Add the configuration to add a javascript version tag to the loaded scripts. Only applied when the Firefox browser is run.

Closes karma-runner#1719.

@wesleyvanderlee
Copy link

It seems I got those errors due to some wrong environment settings in Windows. After exhaustive search for the solution to this problem, I tested your work in a virtual machine, and I'm happy to announce that all tests pass! Furthermore the JavaScript tag does indeed work. Looks good ;).

@Moumi
Copy link
Author

Moumi commented Mar 10, 2016

Since you have had the exact same errors as @liuyiran13 we can safely assume it was also a wrong environment setting. Also, @ludai0106 has confirmed that it works, so I created a PR for this!

@Moumi Moumi force-pushed the feature-javascript-version-tag branch from 19a8ac7 to bc7fd1d Compare March 10, 2016 16:32
@Moumi Moumi force-pushed the feature-javascript-version-tag branch from bc7fd1d to c25ed00 Compare March 20, 2016 20:39
Add the configuration to add a javascript version tag to the loaded scripts. Only applied when the Firefox browser is run.

Closes karma-runner#1719.
@Moumi Moumi force-pushed the feature-javascript-version-tag branch from c25ed00 to 0239c75 Compare March 21, 2016 00:32
@wesleyvanderlee
Copy link

finished

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.

2 participants