It was quick :). Unfortunately mergable does not support commit checking,
but I quickly implemented it in Kaxil's Awesome Boring Cyborg :).

@Kaxil Naik <[email protected]> -> Note that it requires new permission
(Commit Status : read & write). From what I saw adding permissions is a bit
misleading  - after you update the app, you  have to add it in App Settings
and then you can see that the permission was requested and you need to
confirm it before it is effective (but it could be only for development
version).

   - PR to Kaxil's boring-cyborg:
   https://github.com/kaxil/boring-cyborg/pull/2
   - PR in our repo with proper configuration:
   https://github.com/apache/airflow/pull/7149  (to be merged after the
   Cyborg is updated).


J.

On Mon, Jan 13, 2020 at 6:27 AM Jarek Potiuk <[email protected]>
wrote:

> Working on it :)
>
> On Mon, Jan 13, 2020 at 3:50 AM hotmail <[email protected]>
> wrote:
>
>> I just found out some of our PR title have specific `AIRFLOW-XXXX` or
>> `AIRFLOW-\d{4}`, but git commit message don’t, such as
>> https://github.com/apache/airflow/pull/7144 <
>> https://github.com/apache/airflow/pull/7144>, Could we add some bot to
>> check git commit message?
>>
>>
>> Best Wish
>> — Jiajie
>>
>>
>
> --
>
> Jarek Potiuk
> Polidea <https://www.polidea.com/> | Principal Software Engineer
>
> M: +48 660 796 129 <+48660796129>
> [image: Polidea] <https://www.polidea.com/>
>
>

-- 

Jarek Potiuk
Polidea <https://www.polidea.com/> | Principal Software Engineer

M: +48 660 796 129 <+48660796129>
[image: Polidea] <https://www.polidea.com/>

Reply via email to