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 dependency eslint to v6 #2652

Closed
wants to merge 8 commits into from
Closed

Update dependency eslint to v6 #2652

wants to merge 8 commits into from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Jun 22, 2019

This PR contains the following updates:

Package Type Update Change
eslint (source) dependencies major 5.16.0 -> 6.0.1

Release Notes

eslint/eslint

v6.0.1

Compare Source

v6.0.0

Compare Source


Renovate configuration

📅 Schedule: At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻️ Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot. View repository job log here.

@EnTeQuAk
Copy link
Contributor

I started to work on this.

@EnTeQuAk EnTeQuAk mentioned this pull request Jun 26, 2019
@renovate
Copy link
Contributor Author

renovate bot commented Jun 26, 2019

PR has been edited

👷 This PR has received other commits, so Renovate will stop updating it to avoid conflicts or other problems. If you wish to abandon your changes and have Renovate start over you may click the "rebase" checkbox in the PR body/description.

@mantou132
Copy link

Eslint 5 uses esllint-utils 1.4.0
esllint-utils 1.4.0 has cve bug
GHSA-3gx7-xhv7-5mx3

@webmaster128
Copy link

Any updates on this PR? It would be good to get eslint 5 out of the dependency tree

package.json Outdated
@@ -48,7 +48,7 @@
"deepmerge": "3.2.0",
"dispensary": "0.37.0",
"es6-promisify": "6.0.1",
"eslint": "5.16.0",
"eslint": "6.0.1",
Copy link

@webmaster128 webmaster128 Feb 17, 2020

Choose a reason for hiding this comment

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

Could you use the caret notation ^6.0.1, such that we don't need to install multiple versions of eslint 6 in larger projects?

@EnTeQuAk
Copy link
Contributor

I started working on this again and I'm hoping to land this for the Feb 27th milestone.

@diox diox modified the milestones: 2020.02.27, 2020.03.12 Mar 3, 2020
@Rob--W
Copy link
Member

Rob--W commented Apr 16, 2020

eslint 6.0.0 introduces a feature (rfc) that can potentially cause execution of arbitrary code when the linter is run on a third-party project.

Before bumping the version, please verify that this doesn't negatively affect the goals of addons-linter. If needed, set the resolvePluginsRelativeTo option (introduced here (rfc)) to explicitly fix the location of plugins.

@rpl rpl assigned willdurand and unassigned EnTeQuAk Apr 30, 2020
@renovate
Copy link
Contributor Author

renovate bot commented May 8, 2020

Autoclosing Skipped

This PR has been flagged for autoclosing, however it is being skipped due to the branch being already modified. Please close/delete it manually or report a bug if you think this is in error.

@willdurand willdurand removed this from the 2020.03.12 milestone May 25, 2020
@willdurand
Copy link
Member

willdurand commented May 25, 2020

eslint 6.0.0 introduces a feature (rfc) that can potentially cause execution of arbitrary code when the linter is run on a third-party project.

Before bumping the version, please verify that this doesn't negatively affect the goals of addons-linter. If needed, set the resolvePluginsRelativeTo option (introduced here (rfc)) to explicitly fix the location of plugins.

@Rob--W do you think it is a problem given that the linter does not seem to load any eslintrc config?

We still need to load some plugins, and the default value for this new option is CWD so I am not sure what else we could do? Do you think the linter would load a ESLint plugin located in a file/folder of a webext?

@willdurand willdurand closed this Jun 23, 2020
@willdurand willdurand deleted the renovate/eslint-6.x branch June 23, 2020 13:42
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.

7 participants