Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-cloud-dev] [GitHub org] Tracing-related repositories settings
  • From: Marc Dumais <marc.dumais@xxxxxxxxxxxx>
  • Date: Mon, 15 Apr 2024 16:05:46 +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=2IWPKNVsfi5vTK941+0gOFtz76RClLYMQUbi71uFHpE=; b=BtBDblbWpEvdWkya/+W7eK+0h2f4kpXy9G6pdd2zgMUpD+XlOXgE7Q/DcDsvfS5L7CDQ/2lN52Wrsfm1iXEnECODSDagD5S/BeFvPjPq4l255/SoI+sWM7O7rTlb9flhj0OkboiefoKoFsxka8g+JKX41pS1BsVqg8fDZqMTCD2aPmmPA0sdlRRXU80CMFkUlQ3qT1TBwFkAEtXbeoGSQDeEeo8A7CoYkKuXhqoljtoK8igALCAdzrviHdxEILOIW/zciiexopRk1i3sCQzECJMDbR+OM3TMfJKGgsq2zHlesYZnl7FvhHCu+WsxLEHaBiIT3ruDwoEohesHh2p0Nw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZQrxYJ6b4iOxTtVrDnoExJ+Szyszaqf/+uFlMqAgnAQDEixzPeSvPPLCMxj3zmyJoA1juo4kcoRH6JQHAMX4RMRq2eIqz0l6C65rn7kFlgXMs9wIU+VQf1dWtOOZK3uoUoMChEnpH9X3OP6GlSk/vHHAFZGrktR8BlyH+APR0oh/uX1jXF3alTsUZK5VDQJn3wStOZ06/KTqJ9XlzpNbvMxGlKgf5RNl9mt72Oz98cGQhQGk5HrnUsikT7spQmLBF5IBDe9NPuKJYzJ9yBrXCLl6JZfR0Jl2fz/ofzjwWF2N8T1w1AQrgGkNLRfZaMz74yGzEfUCWZLgZnAkefFmIw==
  • Delivered-to: cdt-cloud-dev@xxxxxxxxxxx
  • List-archive: <https://www.eclipse.org/mailman/private/cdt-cloud-dev/>
  • List-help: <mailto:cdt-cloud-dev-request@eclipse.org?subject=help>
  • List-subscribe: <https://www.eclipse.org/mailman/listinfo/cdt-cloud-dev>, <mailto:cdt-cloud-dev-request@eclipse.org?subject=subscribe>
  • List-unsubscribe: <https://www.eclipse.org/mailman/options/cdt-cloud-dev>, <mailto:cdt-cloud-dev-request@eclipse.org?subject=unsubscribe>
  • Msip_labels:
  • Thread-index: AQHajCtBaqMP/+M5sEaqVpp/z5bBwLFpes0pgAAKG0Q=
  • Thread-topic: [GitHub org] Tracing-related repositories settings
Hi Rob,

Unfortunately, our PR [1] was merged about an hour ago, after a lead (Bernd) approved it. Please still have a look and let me know if there's anything you do not agree with - we can discuss and I can do a follow up if necessary.

Regards,
Marc

[1] PR:
https://github.com/eclipse-cdt-cloud/.eclipsefdn/pull/16

From: Rob Moran <Rob.Moran@xxxxxxx>
Sent: Monday, April 15, 2024 11:29 AM
To: cdt-cloud-dev@xxxxxxxxxxx <cdt-cloud-dev@xxxxxxxxxxx>
Cc: Marc Dumais <marc.dumais@xxxxxxxxxxxx>
Subject: Re: [GitHub org] Tracing-related repositories settings
 
I'm keen to check any changes made in this area, but agree some housekeeping could be useful.

I assume any changes will be via PRs we can review and gatekeep as necessary?

Cheers,

Rob
From: cdt-cloud-dev <cdt-cloud-dev-bounces@xxxxxxxxxxx> on behalf of Marc Dumais via cdt-cloud-dev <cdt-cloud-dev@xxxxxxxxxxx>
Sent: 11 April 2024 17:17
To: cdt-cloud-dev@xxxxxxxxxxx <cdt-cloud-dev@xxxxxxxxxxx>
Cc: Marc Dumais <marc.dumais@xxxxxxxxxxxx>
Subject: [cdt-cloud-dev] [GitHub org] Tracing-related repositories settings
 
Hi CDT Cloud devs,

In internal discussions among our small group of contributors, we have concluded that we dislike so-called GitHub "merge commits". We plan to submit a PR towards the organization's ".eclipsefdn" repo [1], to disable such commits for tracing-related repositories, where we are the main maintainers. 

We instead favor merging PRs using "rebase" generally, or "squash" when it's not desired to keep the individual commits that comprise a PR.

While we're there, we will also do a bit of cleanup, for the configs of the same tracing-related repos. e.g. harmonize settings, remove unused webhooks, and potentially other small improvements. 

Any objections?

Thanks,
Marc


[1] https://github.com/eclipse-cdt-cloud/.eclipsefdn
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

Back to the top