When I was updating #153 , I noticed that the autoformatter action and the precommit hook were fighting with each other trying to apply inconsistent prettier changes. That was happening because the workflow was using the latest version of prettier and the precommit hook was using the version specified by package.json.
This PR:
Updates to latest prettier
Modifies the format workflow to use the same commands that the precommit hook does, so future prettier updates won't cause the same class of issue to reappear
Restricts the formatter workflow to only attempt to run on PRs from non-forks, since it would break on forks anyway (it won't have commit permissions)
When I was updating #153 , I noticed that the autoformatter action and the precommit hook were fighting with each other trying to apply inconsistent prettier changes. That was happening because the workflow was using the latest version of prettier and the precommit hook was using the version specified by package.json.
This PR: