Christopher Dignam
Christopher Dignam
If you don't mind adding another GitHub Bot to the mix, I built an [open source bot called Kodiak](http://github.com/chdsbd/kodiak) which supports auto merging pull requests. There's specific instructions for using...
@danielbachhuber Sadly not. There was a discussion about adding that kind of metadata in a structured form via [probot metadata](https://github.com/probot/metadata), but I don't think that's gone anywhere yet. #2294 would...
If you wrap your migration in a transaction squawk won't warn: ```sql begin; create table if not exists customer ( email text ); create unique index if not exists idx_custome_email...
This looks like a bug. We select the first 100 branch protection rules here: https://github.com/chdsbd/kodiak/blob/0065e640d6febdb08ea236f459c125990fd9d9cf/bot/kodiak/queries/__init__.py#L159
The issue in this query is the matchingRefs call: https://github.com/chdsbd/kodiak/blob/ba5073258059fbda248f5ec71eca200da77d50f5/bot/kodiak/queries/__init__.py#L161-L165
I think we report all violations as warnings: https://github.com/sbdchd/squawk/blob/6437c3ca743d07f3c7ce8ebe49240b2c5aa65c1c/cli/src/reporter.rs#L393-L402 Maybe we could mimic [eslint's exit codes](https://eslint.org/docs/latest/use/command-line-interface#exit-codes). - no violations: exit 0 - violations: exit 1 - internal error (parsing error,...
I think this behavior is mimicking the behavior of ESLint. We should add a flag like they have to handle this: https://eslint.org/docs/latest/use/command-line-interface#--no-error-on-unmatched-pattern
I'm not sure that disabling require-concurrent-index-creation when `--assume-in-transaction` is the right decision here. `--assume-in-transaction` simulates linting a migration with a `BEGIN;` `COMMIT;`. I think if a user creates an index,...
We could add another value to the —exclude option, like: backwards-compatibility, locking, schema-style For the documentation website we already define groups like this with tags: https://github.com/sbdchd/squawk/blob/99e93139f430deebc703d0123d398d25fa946550/docs/src/pages/index.js#L105 We could also add...
Hey @qefir, Thank you for the pull request! Can you explain more about your use case? My understanding is you want to disable merging pull requests whenever an issue has...