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: ignore jsVersion configuration property in Firefox 59+ #2958

Merged
merged 1 commit into from
Mar 15, 2018

Conversation

devoto13
Copy link
Collaborator

@devoto13 devoto13 commented Mar 15, 2018

Support for specifying JavaScript version was removed in Firefox 59. See
https://bugzilla.mozilla.org/show_bug.cgi?id=1428745 for more details.
From now on Karma will ignore jsVersion property in Firefox 59+ and
print deprecation warning if jsVersion property is set.

Support for specifying JavaScript version was removed in Firefox 59. See
https://bugzilla.mozilla.org/show_bug.cgi?id=1428745 for more details.
From now on Karma will ignore `jsVersion` property in Firefox 59+ and
print deprecation warning if `jsVersion` property is found.
@jehon
Copy link

jehon commented Mar 15, 2018

This should fix #2957 as soon as it works

@devoto13
Copy link
Collaborator Author

devoto13 commented Mar 15, 2018

Broken build is because new release of karma-qunit broke one of the integration tests. Looks like https://github.com/karma-runner/integration-tests/blob/master/coverage-qunit/package.json should depend on qunit instead of qunitjs now.

Blocked on karma-runner/integration-tests#7

@devoto13 devoto13 closed this Mar 15, 2018
@devoto13 devoto13 reopened this Mar 15, 2018
@dignifiedquire
Copy link
Member

Blocked on karma-runner/integration-tests#7

merged

Copy link
Member

@dignifiedquire dignifiedquire left a comment

Choose a reason for hiding this comment

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

lgtm

@dignifiedquire dignifiedquire merged commit 2694d54 into karma-runner:master Mar 15, 2018
@devoto13 devoto13 deleted the ff59-js-version branch March 15, 2018 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants