Should we add a linting job to the CI, like we have in Purify?

Question

Should we add a linting job to the CI, like we have in Purify?

Opened this issue 2 years ago · 2 comments

Answer 1 · 2023-06-23T15:08:09.000Z

@krishnakumarg1984 I think you did this somewhere but can't remember which branch it was, could you link a PR to this issue when you make it?

Answer 2 · 2023-06-23T15:25:56.000Z

@SJaffa It is here.

Note that, this will create a large diff since, for the first run, the linter will reformat and auto-commit the changed codebase. We have to pull down the reformatted development branch, and merge the changed formatting into each of the PRs to be able to merge again.

Let me know your thoughts?