Page MenuHomeElementl

Disable pre-commit hooks
ClosedPublic

Authored by max on Jul 8 2019, 9:29 PM.
Tags
None
Referenced Files
F2897214: D558.diff
Wed, Mar 29, 12:10 AM
Unknown Object (File)
Wed, Mar 22, 5:32 PM
Unknown Object (File)
Wed, Mar 22, 9:48 AM
Unknown Object (File)
Wed, Mar 22, 9:48 AM
Unknown Object (File)
Wed, Mar 22, 9:48 AM
Unknown Object (File)
Sun, Mar 19, 5:23 AM
Unknown Object (File)
Feb 14 2023, 2:44 PM
Unknown Object (File)
Feb 12 2023, 10:33 AM
Subscribers
None

Details

Summary

These are broken on 2.7 (https://github.com/dagster-io/dagster/issues/1464) and the black
pre-commit hook handles command-line flags in a way that is inconsistent with other invocations --
given that vscode is adequate for day-to-day dev and we also run black in CI, simpler just to
remove.

Test Plan

Manual

Diff Detail

Repository
R1 dagster
Branch
rm-pre-commit-hooks
Lint
No Lint Coverage
Unit
No Test Coverage