Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [theia-dev] Master breakage
  • From: Colin Grant <colin.grant@xxxxxxxxxxxx>
  • Date: Thu, 16 Mar 2023 14:18:26 +0000
  • Accept-language: en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=ericsson.com; dmarc=pass action=none header.from=ericsson.com; dkim=pass header.d=ericsson.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=5utR52zDpwQAr29cIIvkxwLqlRhD3sVXBWt1/UZ0F7w=; b=gTZd+5H1fpc4s8aFr4qVL/w7JyB24OhGaqhAN0KLYf2nJo9sDB3vSkrpBS2SFnwDRxYTwvvCLY717q4POru1X0Yu3KmULR9QbJt1ELUgcMwCpyZ68h5LDKiZQa9EyUkgJdXpQdqm2muYTOb1XyyrEvVpXuAEelLirSPi6XKB4jh2ceFKAnxnJZa1qmAnwkGAwbAZeTLOfkcm+SgCf7amHFy1JiXjw+uP5uo9yKoLdVwt32d/lZG4MgO6C3QdcL2codwBUjiEIkWlA1TlUg8ltHQNbspAD2fnAVZFTOcuR4XuuIAMKnN4J6G6sURsloJtBfiQYdHWEh6mil1+FoBqIw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NiWL5hOJ4CpeIZ6gKBNzzraJxlpoGXddhvRTd252/rGX4NC+h0skfzK0iYMaNBtpF2QNtVvz9GjaL20QMZwFC2zV1McKuMOSnMwIVfZ9sFNzASGxhy9IiFwggC8cjBq9Zi7QuxEin5SdKVuImt9xgRmPloRRScMdofUli3NRHh0DPAc6VV8wl5X3HTDGBKLGanbetB+r/TWjx82fm4ZmfGRl53Dlp904EaReQq4UPjWAfB5GhE5UVOdpp+T9d5icsIFnN2F0pXkb6cg6H+lqGnhznn1dcn2ztsKXR++hi3eFc5qVspehQS0NyhatNp5OAQMC+zBn7L+SD39U+EOWJw==
  • Delivered-to: theia-dev@xxxxxxxxxxx
  • List-archive: <https://www.eclipse.org/mailman/private/theia-dev/>
  • List-help: <mailto:theia-dev-request@eclipse.org?subject=help>
  • List-subscribe: <https://www.eclipse.org/mailman/listinfo/theia-dev>, <mailto:theia-dev-request@eclipse.org?subject=subscribe>
  • List-unsubscribe: <https://www.eclipse.org/mailman/options/theia-dev>, <mailto:theia-dev-request@eclipse.org?subject=unsubscribe>
  • Thread-index: AQHZWBEDvquY6YUTOEyLY9Fnohpr4K79chPw
  • Thread-topic: [theia-dev] Master breakage

Hello all,

 

That was a fairly unusual case of a very old PR that had passed all of the checks at the time and managed not to have any conflicts with the current state of master, but didn’t actually fit the current state of the code. We could go back to a merge strategy that requires manual rebases so that no code that isn’t actually up to date with (and hasn’t run CI with) the current state of master gets in; we could, GitHub actions permitting, invalidate CI runs of a certain age. Both of those would certainly take care of this kind of situation though _always_ requiring a manual rebase on the very tip of master would like be a pain. If it’s possible, I’d lean towards some sort of staleness in CI, but fortunately, with our drive to clear out the PR queue, we also don’t have that many old PR’s sitting around in this condition.

 

Best,

Colin

 

From: theia-dev <theia-dev-bounces@xxxxxxxxxxx> On Behalf Of Thomas Mäder
Sent: Thursday, March 16, 2023 8:10 AM
To: theia-dev@xxxxxxxxxxx
Subject: [theia-dev] Master breakage

 

Hi folks,

 

we recently merged https://github.com/eclipse-theia/theia/pull/10338, which seems to have broken master. Mistakes happen, but I'm wondering if we can improve our process? Did the linter check fail and we merged anyway, or is there some technical improvement we can make?

 

I'd also would encourage anyone who is involved in a "master is broken"-situation to write a quick mail to this very list to alert the community. It keeps everyone with failing PR's from having to investigate what's wrong themselves.

 

thanks,

 

/Thomas

Back to the top