From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by mx.groups.io with SMTP id smtpd.web12.9196.1575392878708013609 for ; Tue, 03 Dec 2019 09:07:58 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.93, mailfrom: michael.d.kinney@intel.com) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 03 Dec 2019 09:07:58 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.69,273,1571727600"; d="scan'208";a="235952132" Received: from orsmsx104.amr.corp.intel.com ([10.22.225.131]) by fmsmga004.fm.intel.com with ESMTP; 03 Dec 2019 09:07:58 -0800 Received: from orsmsx154.amr.corp.intel.com (10.22.226.12) by ORSMSX104.amr.corp.intel.com (10.22.225.131) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 3 Dec 2019 09:07:57 -0800 Received: from orsmsx113.amr.corp.intel.com ([169.254.9.200]) by ORSMSX154.amr.corp.intel.com ([169.254.11.18]) with mapi id 14.03.0439.000; Tue, 3 Dec 2019 09:07:56 -0800 From: "Michael D Kinney" To: Laszlo Ersek , "devel@edk2.groups.io" , Sean Brogan , "Bret Barkelew" , "Gao, Liming" , "Kinney, Michael D" Subject: Re: [edk2-devel] EDK II Maintainers - EDK II CI is now active on edk2/master Thread-Topic: [edk2-devel] EDK II Maintainers - EDK II CI is now active on edk2/master Thread-Index: AdWY/xjw4dZ1iskvSlK5J8Hop+ZpewLdrbAAADbisuAANUr3gADIaoHwACyZ/wAAAAenEA== Date: Tue, 3 Dec 2019 17:07:56 +0000 Message-ID: References: <545f1da9-a998-1095-57f9-085cbd535596@redhat.com> <184dbbde-0916-7e09-476c-de700cc57dc5@redhat.com> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action x-originating-ip: [10.22.254.140] MIME-Version: 1.0 Return-Path: michael.d.kinney@intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi Laszlo, I can think of ways we may be able to get the label=20 information into the body of the email in a PR comment. I will have to see if there is anyway to adjust the subject of the email. The subject appears to always match the title of the PR. That title is editable, but may only be editable using GitHub APIs. Mergify does not support that feature=20 today. If you look at the following file, you will see the current Mergify rules. https://github.com/tianocore/edk2/blob/master/.mergify/config.yml If I look at the 3rd rule for merge conflicts, I could split that out into 2 rules. One for push label set and one for push label not set and adjust the message generated to state if the conflict is on a personal build or a push request. Would small adjustments like this in the body of the email notifications help? If we make the generated messages consistent email filters on the content of the email body can be used=20 instead of email filters on the email subject. Each developer can enable/disable the 'Watch' feature on the edk2 repo. That is a developer setting, not a team setting. Mike > -----Original Message----- > From: Laszlo Ersek > Sent: Tuesday, December 3, 2019 12:56 AM > To: Kinney, Michael D ; > devel@edk2.groups.io; Sean Brogan > ; Bret Barkelew > ; Gao, Liming > > Subject: Re: [edk2-devel] EDK II Maintainers - EDK II > CI is now active on edk2/master >=20 > On 12/02/19 20:55, Kinney, Michael D wrote: >=20 > > So my question still stands. What notifications > would > > you like to see if we have the use case of a single > PR > > with multiple rounds of reviews and a transition from > > a PR without the 'push' label to a PR with a 'push' > label? >=20 > Thank you for explaining. >=20 > In this case, I think it would be helpful if, whenever > another email > notification were sent out about a PR, the subject of > that email > contained an auto-generated part that advertized the > push label's > presence on the PR, at that time. >=20 > OTOH... It's also possible that I'm approaching this > wrong. After all, > before we adopted github.com for pushing, a maintainer > would just go > ahead and push post-review (with no *automatic* email > notification about > the fact), and then we'd expect that maintainer to > report back under the > patch review thread ("pushed as commit range ..."), > and/or close the BZ > ticket with a similar comment. (And then there would be > an > auto-generated bugzilla email about that.) >=20 > My point being, maybe I shouldn't even *read* these > github.com > notifications at all. I'm not sure if they give me > useful information. > (When the PR is ultimately merged, we *still* require > the above kind of > closing comment in Bugzilla, from the maintainer -- > thus the originally > needed information is still provided, just like > before.) >=20 > It's just that, *if* I attempt to read the github.com > emails, *then* > they confuse me (for example because they don't expose > the push label). > If we consider this specific kind of PR that we have > adopted for edk2 a > (practically) "maintainer-internal", mechanical, > action, then I guess I > might as well want to unsubscribe from those > notifications. After all > I'm still subscribed to BZ emails, and I'll see the > resultant commit > range noted there (through the comment that the > assignee adds manually, > when they close the BZ). >=20 > Is it possible for a tianocore group member to > "unwatch" PRs? >=20 > Thanks! > Laszlo