Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

chore: Update eslint-utils#301

Merged
ajmeese7 merged 1 commit into
nodejs:masterfrom
Trott:eslint-utils-update
Sep 1, 2019
Merged

chore: Update eslint-utils#301
ajmeese7 merged 1 commit into
nodejs:masterfrom
Trott:eslint-utils-update

Conversation

@Trott

@Trott Trott commented Aug 26, 2019

Copy link
Copy Markdown
Member

Description

Update eslint-utils in two .lock files. Currently-used version is
susceptible to a security vulnerability.

Related Issues

Comment thread gatsby-config.js
url: `${site.siteMetadata.siteUrlNoSlash}/${
edge.node.fields.slug
}`,
url: `${site.siteMetadata.siteUrlNoSlash}/${edge.node.fields.slug}`,

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This seemingly-unrelated change made because the prettier check complained and wouldn't let me push changes to the repo.

@codecov-io

codecov-io commented Aug 26, 2019

Copy link
Copy Markdown

Codecov Report

Merging #301 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #301   +/-   ##
=======================================
  Coverage   39.92%   39.92%           
=======================================
  Files          22       22           
  Lines         273      273           
  Branches       62       62           
=======================================
  Hits          109      109           
  Misses        162      162           
  Partials        2        2

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eaa1664...19bdfa4. Read the comment docs.

Update eslint-utils in two .lock files. Currently-used version is
susceptible to a security vulnerability.
Comment thread gatsby-config.js
query: `{
site {
siteMetadata {
siteUrl,

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

New gatsby errors without this.

@Trott

Trott commented Aug 26, 2019

Copy link
Copy Markdown
Member Author

/gcbrun

@Trott

Trott commented Aug 29, 2019

Copy link
Copy Markdown
Member Author

@nodejs/nodejs-dev This needs a review.

@Trott

Trott commented Sep 1, 2019

Copy link
Copy Markdown
Member Author

@nodejs/website-redesign @nodejs/website This could use a review.

@ajmeese7 ajmeese7 merged commit 2a806ef into nodejs:master Sep 1, 2019
jonchurch pushed a commit to jonchurch/nodejs.dev that referenced this pull request Sep 17, 2019
Update eslint-utils in two .lock files. Currently-used version is
susceptible to a security vulnerability.
@Trott Trott deleted the eslint-utils-update branch January 18, 2022 19:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants