Skip to content
This repository was archived by the owner on May 17, 2024. It is now read-only.

Conversation

dlawin
Copy link
Contributor

@dlawin dlawin commented Nov 3, 2023

Currently, review dog will add suggestions, but the step is always green. Update to fail the CI job on bad formatting

@dlawin dlawin force-pushed the block_merge_on_bad_formatting branch from 6286e6f to 22fe539 Compare November 3, 2023 22:02
@dlawin dlawin changed the title test blocking merge on formatter error Block merge on formatter error Nov 3, 2023
@dlawin dlawin requested a review from sungchun12 November 3, 2023 22:18
@dlawin dlawin self-assigned this Nov 3, 2023
@sungchun12 sungchun12 force-pushed the block_merge_on_bad_formatting branch from 22fe539 to 90c048b Compare November 4, 2023 19:51
@sungchun12 sungchun12 merged commit 886a615 into master Nov 4, 2023
@sungchun12 sungchun12 deleted the block_merge_on_bad_formatting branch November 4, 2023 20:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants