search

alan-turing-institute / rctab-cli

RCTab CLI
MIT License
0 stars 0 forks source link

Move safety hook to own config #18

Closed Iain-S closed 2 months ago

Iain-S commented 2 months ago

Summary

It effectively splits pre-commit run into pre-commit run --config old-config-but-without-safety.yaml and pre-commit run config-with-only-safety.yaml. We don't want to have to fix safety warnings every PR merge as we would like PRs to be focused on a single thing.

Issues

Reviewer

How to run

  1. Open a PR.
  2. See what happens.
dlpbc commented 2 months ago

We can merge without all checks passing here as it was resolved in #20