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

Update karma to address vulnerabilities. #4038

Merged
merged 10 commits into from
Aug 2, 2019
Merged

Update karma to address vulnerabilities. #4038

merged 10 commits into from
Aug 2, 2019

Conversation

Fawke
Copy link
Contributor

@Fawke Fawke commented Jul 30, 2019

Type of change

  • Build related changes
  • CI related changes

Description of change

Related to #3966. The earlier PR was reverted because Circle CI failed. This PR attempts to fix the failing CI along with the necessary updates required to keep vulnerabilities to a minimum.

The updated package karma v^4.2.0 uses object spread operator which is supported only in Node v9 or greater, hence this PR updates minimum supported Node version to 9.

@Fawke
Copy link
Contributor Author

Fawke commented Jul 30, 2019

Update - command gulp serve is currently exiting out because of this change, looking into the issue.

@jsnellbaker jsnellbaker self-requested a review July 30, 2019 12:56
@jsnellbaker jsnellbaker self-assigned this Jul 30, 2019
@Fawke
Copy link
Contributor Author

Fawke commented Jul 31, 2019

Update - command gulp serve is currently exiting out because of this change, looking into the issue.

@jsnellbaker I've fixed this. The command gulp serve now works as earlier without exiting.

@jsnellbaker jsnellbaker added LGTM needs 2nd review Core module updates require two approvals from the core team and removed needs review labels Jul 31, 2019
Copy link
Collaborator

@snapwich snapwich left a comment

Choose a reason for hiding this comment

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

LGTM

@jsnellbaker jsnellbaker merged commit 2fa4387 into master Aug 2, 2019
sa1omon pushed a commit to gamoshi/Prebid.js that referenced this pull request Nov 28, 2019
* update packages to fix vulns

* upgrade node version to support advanced features

* fix command gulp test --browserstack not exiting

* fixes gulp view-coverage command failing

* fix: command, gulp serve now keeps running without exiting

* fix: error in package-lock.json

* update README.md to reflect change in node version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LGTM needs 2nd review Core module updates require two approvals from the core team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants