chore: add `format` workflow

eslint-community / eslint-plugin-security

ESLint rules for Node Security

Apache License 2.0

2.22k stars 109 forks source link

chore: add `format` workflow #120

Closed MichaelDeBoey closed 1 year ago

nzakas commented 1 year ago

Yeah, I'm not a fan of this type of workflow. We are already formatting on precommit, and I think that's enough.

MichaelDeBoey commented 1 year ago

@nzakas There's nothing enforcing it though. If people disable the precommit hook, it won't happen. This PR makes sure it's enforced in a way that it's not holding back PRs

nzakas commented 1 year ago

I just don't think formatting is important enough to go through this extra effort. I appreciate the suggestion but it's not the way I prefer to run repos.