From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web12.2611.1587631350271542988 for ; Thu, 23 Apr 2020 01:42:30 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: ard.biesheuvel@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id DF01F30E; Thu, 23 Apr 2020 01:42:28 -0700 (PDT) Received: from [10.37.8.121] (unknown [10.37.8.121]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id DBE543F73D; Thu, 23 Apr 2020 01:42:26 -0700 (PDT) Subject: Re: [EXTERNAL] Re: [PATCH v2 0/6] Edk2 Platform and Core CI for ArmVirtPkg, EmulatorPkg, and OvmfPkg To: Sean Brogan , Laszlo Ersek , "michael.kubacki@outlook.com" , "devel@edk2.groups.io" Cc: Andrew Fish , Bret Barkelew , Jordan Justen , Leif Lindholm , Liming Gao , "Kinney, Michael D" , Ray Ni References: <5fcd80ac-03d1-a806-6a69-4602bc0d352c@redhat.com> From: "Ard Biesheuvel" Message-ID: <1625ea2f-ebf3-7f1e-fc42-efa0ef370923@arm.com> Date: Thu, 23 Apr 2020 10:42:25 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit On 4/23/20 2:45 AM, Sean Brogan wrote: > I was hoping that because it wasn't "HTML tag soup" that build status could be front and center in the package readme as I find that more in line with expectations on github based projects. Nesting it deeper in the package just means less people find it when looking at your package. But I could see a few ways to solve it so I am looking for feedback and alignment before doing anything else. > > Option 1. > > 1. Add the Platform CI build status for the three platforms to the Edk2 repo readme since these are platforms in edk2 this makes sense and brings the most visibility to their status and the existence of the builds. > 2. Convert edk2 repo readme to RST so it avoids "HTML tag soup" > 3. Move the remaining part of the OvmfPkg/Readme.rst file into OvmfPkg/PlatformCI/Readme.md (I prefer MD when not doing large tables of links as it is easier to write and slightly easier to read as plain text). > 4. Do step 3 for ArmVirtPkg and EmulatorPkg too > > Option 2. > 1. Leave the ReadMe.rst at the root of each package but only include the build status table and add a link to the PlatformCi/ReadMe.md > 2. Move the Pytool and Platform CI focused readme contents to PlatformCI/ReadMe.md > > Option 3. > > Something else. > Given the intent to make it a requirement that changes don't regress on these platforms we've selected, I think having the CI build (and boot!) status of these platforms front and center makes sense. I am absolutely fine with using either .md or .rst depending on the needs of the individual use cases. So I suppose that means option #1, or some variation thereof. > Finally does anyone have a strong preference for ReadMe vs README vs readme vs Readme vs ReAdMe? I see a mix of these in the edk2 code tree. > ReadMe is the most idiomatic given the EDK2 coding style, but I could live with README as well.