Lint error when building assets

[fauria]

When running gulp build or grunt build, trailing commas are added to config/assets/production.js resulting in:

...
Running "eslint:target" (eslint) task

/Directory/config/assets/production.js
  11:65  error  Unexpected trailing comma  comma-dangle
  28:60  error  Unexpected trailing comma  comma-dangle

✖ 2 problems (2 errors, 0 warnings)

Warning: Task "eslint:target" failed. Use --force to continue.

Edit: Only gulp build adds those commas, grunt buildworks fine.

[shanavas786]

Its because grunt build just use the default assets configurations and doesn't create one based on bower.json (No wiredep task. Porbably a bug :) ).

I have worked on removing trailing comma using gulp-trailing-comma. But the npm package is not up-to-date friskfly/gulp-trailing-comma#2. I will create a PR once it is updated.

[ilanbiala]

Is there still work that needs to be done here?

[shanavas786]

Since there is no update on friskfly/gulp-trailing-comma#2 , i have created a package and plugins for gulp and grunt for removing trailing commas.

[ilanbiala]

@lirantal can you take a look at the PR?

[lirantal]

Re-opening this issue as it is still relevant, waiting for @shanavas786 PR.

[shanavas786]

I have started #1412 .
@lirantal Please have a look.

[mleanos]

@shanavas786 @lirantal Moving to backlog since there hasn't been a successful PR that solves this issue.