You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Following devrel issue 135, this issue has a JavaScript or TypeScript linter being added to each of the projects in this repo (three projects at the time of this writing), and being a required check on all builds.
The author recommends eslint with a linter configuration in the package.json similar to this. The eslint-config-airbnb-base is quite picky so there may be many rules that can and should be disabled. The author recommends restricting rules that cause a linter error (and thus a build failure) only to rules that possibly indicate a mistake. All other rules should be warnings or disabled. The only linter issues that should truly block a merge or a deployment should be linter issues that may genuinely indicate a mistake with repercussions.
Following devrel issue 135, this issue has a JavaScript or TypeScript linter being added to each of the projects in this repo (three projects at the time of this writing), and being a required check on all builds.
The author recommends eslint with a linter configuration in the
package.json
similar to this. Theeslint-config-airbnb-base
is quite picky so there may be many rules that can and should be disabled. The author recommends restricting rules that cause a linter error (and thus a build failure) only to rules that possibly indicate a mistake. All other rules should be warnings or disabled. The only linter issues that should truly block a merge or a deployment should be linter issues that may genuinely indicate a mistake with repercussions.See Also
The text was updated successfully, but these errors were encountered: