Skip to content
This repository was archived by the owner on Aug 6, 2021. It is now read-only.

Linting#22

Merged
eeertekin merged 3 commits intojotform:masterfrom
ctsstc:linting
Feb 28, 2018
Merged

Linting#22
eeertekin merged 3 commits intojotform:masterfrom
ctsstc:linting

Conversation

@ctsstc
Copy link
Contributor

@ctsstc ctsstc commented Feb 28, 2018

Changes

  • Adds eslint magic
  • Quick refactor of some unused variables/globals and a quick refactor to use an unused variable.

Notes

Updates grunt-contrib-jshint due to a bug with building.

See https://goo.gl/oUmN6c
Removes some unused variables and globals.
Refactors one unused variable to be used.

Generated eslintrc.json with `eslint --init` with the following config:

```
? How would you like to configure ESLint? Inspect your JavaScript file(s)
? Which file(s), path(s), or glob(s) should be examined? css.js
? What format do you want your config file to be in? JSON
? Are you using ECMAScript 6 features? No
? Where will your code run? Browser
? Do you use CommonJS? Yes
? Do you use JSX? No
```
@eeertekin eeertekin merged commit a92919f into jotform:master Feb 28, 2018
@eeertekin
Copy link
Contributor

Thank you @ctsstc

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants