Skip to content

Commit

Permalink
Change dangling comma policy (#18)
Browse files Browse the repository at this point in the history
* Always require comma dangle on multiline
* Change version to 7, as the rule change will break existing builds
  • Loading branch information
archinal authored Feb 2, 2018
1 parent 90e2b45 commit 888c37d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "fs-default-project-config",
"version": "6.1.3",
"version": "7.0.0",
"description": "Repo containing commons fox sports project linting configs and editor settings",
"main": "index.js",
"scripts": {
Expand Down
2 changes: 1 addition & 1 deletion resources/.eslintrc-standards.js
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ module.exports = {
"brace-style": ["error", "1tbs", {"allowSingleLine": true}], // enforce consistent brace style for blocks
"camelcase": ["error", {properties: "never"}], // enforce camelcase naming convention
"capitalized-comments": "off", // enforce or disallow capitalization of the first letter of a comment
"comma-dangle": "error", // require or disallow trailing commas
"comma-dangle": ["error", "always-multiline"], // require or disallow trailing commas
"comma-spacing": "error", // enforce consistent spacing before and after commas
"comma-style": "error", // enforce consistent comma style
"computed-property-spacing": "error", // enforce consistent spacing inside computed property brackets
Expand Down

0 comments on commit 888c37d

Please sign in to comment.