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.

Sign up for GitHub

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

Jump to bottom

Update modules to use ESlint semistandard #432

Closed
44 tasks done
ekryski opened this issue Oct 17, 2016 · 3 comments
Closed
44 tasks done

Update modules to use ESlint semistandard #432

ekryski opened this issue Oct 17, 2016 · 3 comments
Milestone
Buzzard

Comments

@ekryski
Copy link
Contributor

ekryski commented Oct 17, 2016 •
edited by daffl
Loading

For each repo we need to:

  • remove .jshintrc
  • update the .gitignore and .npmignore files
  • update package.json file
    • add a lint task
    • install semistandard
    • install eslint-if-supported (if repo supports < node v4)
    • remove any inline jshint directives
    • Add the semistandard config:
  "semistandard": {
    "env": [
      "mocha"
    ],
    "ignore": [
      "/lib"
    ]
  },

Repos
@ekryski ekryski added the Task label Oct 17, 2016
@ekryski ekryski modified the milestone: Buzzard Oct 17, 2016
@marshallswain
Copy link
Member

Instead of a semistandard npm script, let's just use lint, so it's future proof.

@ekryski ekryski mentioned this issue Oct 17, 2016
Closed
18 tasks
@corymsmith corymsmith self-assigned this Oct 22, 2016
@daffl
Copy link
Member

daffl commented Feb 9, 2017

I think we got them all now.

@daffl daffl closed this as completed Feb 9, 2017
@lock
Copy link

lock bot commented Feb 7, 2019

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue with a link to this issue for related bugs.

@lock lock bot unassigned corymsmith Feb 7, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Feb 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Buzzard
Development

No branches or pull requests
4 participants
@corymsmith @marshallswain @daffl @ekryski