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

add ESLint-plugin-Meteor #540

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
1 participant
@dferber90
Copy link
Contributor

commented Mar 10, 2016

This adds ESLint-plugin-Meteor.

Changes

  • adds ESLint-plugin-Meteor
  • upgrade ESLint to v2
  • use Meteor environment, remove resulting duplicate globals definitions
  • enable ESLint-plugin-Meteor in eslintrc
  • fix surfaced issues

Potential Problems
I haven't tested the fixes for the surfaced issues. After pulling/merging this branch, make sure to run npm install again.

I know ESLint-plugin-Meteor has been added before and then I dropped support for it because ESLint and Meteor were both changing so heavily and it wasn't possible to keep up. Now things have settled down and I rewrote the plugin.


To read up on linting and ESLint-plugin-Meteor, you can read this article.


This change is Review on Reviewable

add ESLint-plugin-Meteor
- adds ESLint-plugin-Meteor
- upgrade ESLint to v2
- use Meteor environment, remove resulting duplicate globals definitions
- enable ESLint-plugin-Meteor in eslintrc

@dferber90 dferber90 closed this Jul 27, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.