Re: Loosening the commit limit for work branches

2022-08-26 Thread Harald Sitter
On Thu, Aug 25, 2022 at 11:44 AM Ben Cooksley wrote: > > On Thu, Aug 25, 2022 at 9:27 PM Harald Sitter wrote: >> >> On Wed, Aug 24, 2022 at 8:11 PM Ben Cooksley wrote: >> > >> > On Thu, Aug 25, 2022 at 2:16 AM Harald Sitter wrote: >> >> >> >> On Wed, Aug 24, 2022 at 3:31 PM Noah Davis wrote:

Re: Loosening the commit limit for work branches

2022-08-25 Thread Ben Cooksley
On Thu, Aug 25, 2022 at 9:27 PM Harald Sitter wrote: > On Wed, Aug 24, 2022 at 8:11 PM Ben Cooksley wrote: > > > > On Thu, Aug 25, 2022 at 2:16 AM Harald Sitter wrote: > >> > >> On Wed, Aug 24, 2022 at 3:31 PM Noah Davis wrote: > >> > > >> > A week ago, I ran into an unexpected issue while

Re: Loosening the commit limit for work branches

2022-08-25 Thread Ben Cooksley
On Thu, Aug 25, 2022 at 7:13 AM David Hurka wrote: > On Wednesday, August 24, 2022 8:11:14 PM CEST Ben Cooksley wrote: > > The limitation is aligned with the maximum number of new commits you are > > allowed to introduce to a standard branch. > > We have those limits because the commit hooks

Re: Loosening the commit limit for work branches

2022-08-25 Thread Harald Sitter
On Wed, Aug 24, 2022 at 8:11 PM Ben Cooksley wrote: > > On Thu, Aug 25, 2022 at 2:16 AM Harald Sitter wrote: >> >> On Wed, Aug 24, 2022 at 3:31 PM Noah Davis wrote: >> > >> > A week ago, I ran into an unexpected issue while working on a QML port >> > of Spectacle. There is an undocumented

Re: Loosening the commit limit for work branches

2022-08-25 Thread Halla Rempt
On woensdag 24 augustus 2022 17:12:51 CEST Milian Wolff wrote: > sooner or later you should hit a small mile stone that you can merge already Um, no? Like, no way, not at all? As in quite often, a rewrite is way bigger than that. I merged one this year that took three years of coding...

Re: Loosening the commit limit for work branches

2022-08-24 Thread Milian Wolff
On Mittwoch, 24. August 2022 18:57:12 CEST Noah Davis wrote: > I didn't want to leave the master branch in a semi-broken state or > present what I had come up with in an MR until I could present it as > an improvement. It took a while to figure out what I wanted and > whether some of what I wanted

Re: Loosening the commit limit for work branches

2022-08-24 Thread Ben Cooksley
On Thu, Aug 25, 2022 at 2:16 AM Harald Sitter wrote: > On Wed, Aug 24, 2022 at 3:31 PM Noah Davis wrote: > > > > A week ago, I ran into an unexpected issue while working on a QML port > > of Spectacle. There is an undocumented pre-receive hook that sets a > > 100 commit limit for all branches

Re: Loosening the commit limit for work branches

2022-08-24 Thread Noah Davis
I didn't want to leave the master branch in a semi-broken state or present what I had come up with in an MR until I could present it as an improvement. It took a while to figure out what I wanted and whether some of what I wanted was even achievable. I haven't been working alone, so some people

Re: Loosening the commit limit for work branches

2022-08-24 Thread Milian Wolff
On Mittwoch, 24. August 2022 17:26:33 CEST Noah Davis wrote: > On Wed, Aug 24, 2022 at 5:12 PM Milian Wolff wrote: > > Without any knowledge of your work on the QML port of Spectacle, I would > > also claim that there is bound to be a lot of generic work that should be > > possible to land

Re: Loosening the commit limit for work branches

2022-08-24 Thread Noah Davis
On Wed, Aug 24, 2022 at 5:12 PM Milian Wolff wrote: > Without any knowledge of your work on the QML port of Spectacle, I would also > claim that there is bound to be a lot of generic work that should be possible > to land directly in the main branch, no? You are probably splitting up the > data

Re: Loosening the commit limit for work branches

2022-08-24 Thread Milian Wolff
On Mittwoch, 24. August 2022 15:30:06 CEST Noah Davis wrote: > A week ago, I ran into an unexpected issue while working on a QML port > of Spectacle. There is an undocumented pre-receive hook that sets a > 100 commit limit for all branches of official repos, including work > branches. The error

Re: Loosening the commit limit for work branches

2022-08-24 Thread Harald Sitter
On Wed, Aug 24, 2022 at 3:31 PM Noah Davis wrote: > > A week ago, I ran into an unexpected issue while working on a QML port > of Spectacle. There is an undocumented pre-receive hook that sets a > 100 commit limit for all branches of official repos, including work > branches. The error message it