Re: [PATCH v3 2/3] CI: limit GitHub Actions to designated branches
- From
Jeff King <peff@peff.net>
- Date
- May 5, 2020, 17:11 UTC
- Message-ID
- <20200505171108.GF64077@coredump.intra.peff.net>
- In-Reply-To
- <20200505170546.GD927@danh.dev>
On Wed, May 06, 2020 at 12:05:46AM +0700, Đoàn Trần Công Danh wrote:
Show 12 quoted lines
> > Should this be "for-ci/**", or are we intending for "for-ci-foo" to > > work? I'd suspect anybody who uses this would use a full directory > > namespace in a refspec (like "refs/heads/*:refs/heads/for-ci/*"). It > > might be simpler conceptually to only support that. > > I made this because I saw someone mentioned that they would like to > push to 'for-ci' and expect it works for them. > > I guess it may be better to have: > > - for-ci > - for-ci/**
Ah, that makes sense.
Yeah, the double-rule looks fine to me, but understanding that use, the original "for-ci**" is OK to me, too.
-Peff