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

Npm #191

Merged
merged 2 commits into from
Jul 8, 2020
Merged

Npm #191

merged 2 commits into from
Jul 8, 2020

Conversation

brettz9
Copy link
Contributor

@brettz9 brettz9 commented May 19, 2020

What is the purpose of this pull request?

  • Documentation update

  • Bug fix

  • New rule

  • Changes an existing rule

  • Add autofixing to a rule

  • Other, please explain:

  • package.json devDependency updates and property addition/changes

  • Travis CI-related changes.

What changes did you make? (Give an overview)

  • Travis: Drop end-of-lifed Node 6 and 8, and add Node 12 and 14
  • npm: Update lint-staged per update (advanced configs removed and git add handled automatically)
  • npm: Update devDeps. (all available except eslint (since some plugins still not ready) and prettier (since style changes still under discussion))
  • npm: add recommended package.json properties (also used on npm) (Added bugs, homepage, dependencies
  • npm: Make repository a URL (can be cmd-opened in IDEs)

Avoids Prettier updates as discussed in #185 (though the prettier changes are still on #184 though now rebased on this current PR) and avoids the eslint update (which is covered in #190).

- npm: Update lint-staged per update (advanced configs removed and git add handled automatically)
- npm: Update devDeps. (all available except eslint (since some plugins still not ready) and prettier (since style changes still under discussion))
- Added `bugs`, `homepage`, `dependencies`
- Also made `repository` a URL (can be cmd-opened in IDEs)
@brettz9 brettz9 mentioned this pull request May 19, 2020
6 tasks
@xjamundx xjamundx merged commit 252dd8c into eslint-community:master Jul 8, 2020
@xjamundx
Copy link
Contributor

xjamundx commented Jul 8, 2020

Seems cool. WIll merge.

@brettz9 brettz9 deleted the npm branch July 8, 2020 18:46
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.

None yet

2 participants