Skip to content
This repository has been archived by the owner on May 2, 2022. It is now read-only.

Update eslint globals #181

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

cjkoepke
Copy link

@cjkoepke cjkoepke commented Mar 20, 2020

Description of the Change

Benefits

Avoids linting errors on initial commit from husky.

Verification Process

  • Make a simple change and do a test commit (or run husky manually). It should not flag errors in the webpack config files.

Checklist:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests passed.

Applicable Issues

  • Linting errors on webpack config files on initial commit and/or husky operation.

Changelog Entry

  • Fix ESLINT errors on husky operation for webpack config files.

Copy link
Member

@nicholasio nicholasio left a comment

Choose a reason for hiding this comment

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

Do you mind changing .eslintc to .eslintrc.js since the former has been deprecated?

@cjkoepke
Copy link
Author

@nicholasio From their docs

use a JavaScript, JSON or YAML file to specify configuration information for an entire directory and all of its subdirectories

I updated it to use .eslintrc.json does that work?

@nicholasio
Copy link
Member

@cjkoepke yes, that's fine.

@timwright12
Copy link
Contributor

@cjkoepke can you submit a PR to dover this issue in the plugin repo as well? I filed the issue you for here: 10up/plugin-scaffold#114 - we need to keep those repos sync'd up.

Copy link
Contributor

@timwright12 timwright12 left a comment

Choose a reason for hiding this comment

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

Update plugin repo as well

@jeffpaul
Copy link
Member

@cjkoepke are you able to mimic this PR over in the plugin repo?

@cjkoepke
Copy link
Author

@jeffpaul Yeah, I'll get to this as soon as I can.

@cjkoepke
Copy link
Author

@jeffpaul @timwright12 I've replicated this update on the plugin scaffold: 10up/plugin-scaffold#125

@timwright12
Copy link
Contributor

@cjkoepke looks like we have some conflicts

@jeffpaul jeffpaul requested review from joesnellpdx and removed request for timwright12 November 13, 2020 16:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants