djencks commented on pull request #3912: URL: https://github.com/apache/camel/pull/3912#issuecomment-644241879
I think it would be better to have this in the .eslintrc file: /*eslint semi: ["error", "never"]*/ Can we remove the unused import here instead of suppressing the warning? /* eslint-disable-next-line no-unused-vars */ Unfortunately there's a merge conflict now, can you rebase on master? I think it might be clearer (if you can figure out how :-) to have the added lint-config files in one commit and the effect of running lint in a separate commit. I've found this useful for splitting commits: https://hisham.hm/2019/02/12/splitting-a-git-commit-into-one-commit-per-file/ ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
