refactor: always add trailing comma when multiline (#4)

.travis.yml

@@ -1,7 +1,6 @@
 sudo: false
 language: node_js
 node_js:
- - '6'
  - '8'
  - '10'
 install:

appveyor.yml

@@ -1,6 +1,5 @@
 environment:
  matrix:
- - nodejs_version: '6'
  - nodejs_version: '8'
  - nodejs_version: '10'
 

index.js

@@ -202,12 +202,7 @@ module.exports = {
 
  // eslint eslint#comma-dangle
  'trailing-comma': [ true, {
- multiline: {
- objects: 'always',
- arrays: 'always',
- functions: 'never',
- typeLiterals: 'ignore',
- },
+ multiline: 'always',
  }],
 
  // eslint#no-cond-assign

package.json

@@ -34,9 +34,9 @@
  "homepage": "https://github.com/eggjs/tslint-config-egg",
  "author": "whxaxes <whxaxes@gmail.com>",
  "engines": {
- "node": ">=6.0.0"
+ "node": ">=8.0.0"
  },
  "ci": {
- "version": "6, 8, 10"
+ "version": "8, 10"
  }
 }

test/fixtures/align/parameters.ts

@@ -1,4 +1,4 @@
 function bar(
  a: string,
- b: string
+ b: string,
 ) {}

test/fixtures/comma/trailing-comma-error.ts

@@ -3,9 +3,24 @@ export const foo = {
  b: 2
 };
 
+export const bar = [
+ 1,
+ 2
+];
+
 function foo2(
  a: string,
- b: string,
+ b: string
 ) {
  console.info(a + b);
 }
+
+import {
+ a,
+ b
+} from 'egg';
+
+export {
+ a,
+ b
+} from 'egg';

test/fixtures/comma/trailing-comma.ts

@@ -3,9 +3,24 @@ export const foo = {
  b: 2,
 };
 
+export const bar = [
+ 1,
+ 2,
+];
+
 function foo2(
  a: string,
- b: string
+ b: string,
 ) {
  console.info(a + b);
 }
+
+import {
+ a,
+ b,
+} from 'egg';
+
+export {
+ a,
+ b,
+} from 'egg';

test/index.test.js

@@ -17,7 +17,6 @@ describe('test/index.test.js', () => {
  return coffee.spawn('tslint', [ path.resolve(cwd, './trailing-comma-error.ts') ])
  // .debug()
  .expect('stdout', /Missing trailing comma/)
- .expect('stdout', /Unnecessary trailing comma/)
  .expect('code', 2)
  .end();
  });