From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by mx.groups.io with SMTP id smtpd.web09.1959.1572496959078441602 for ; Wed, 30 Oct 2019 21:42:39 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.120, 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 fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 30 Oct 2019 21:42:38 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.68,250,1569308400"; d="scan'208";a="225565877" Received: from orsmsx106.amr.corp.intel.com ([10.22.225.133]) by fmsmga004.fm.intel.com with ESMTP; 30 Oct 2019 21:42:38 -0700 Received: from orsmsx158.amr.corp.intel.com (10.22.240.20) by ORSMSX106.amr.corp.intel.com (10.22.225.133) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 30 Oct 2019 21:42:38 -0700 Received: from orsmsx113.amr.corp.intel.com ([169.254.9.28]) by ORSMSX158.amr.corp.intel.com ([169.254.10.56]) with mapi id 14.03.0439.000; Wed, 30 Oct 2019 21:42:37 -0700 From: "Michael D Kinney" To: Leif Lindholm , "Kinney, Michael D" CC: "devel@edk2.groups.io" , Andrew Fish , Laszlo Ersek Subject: Re: [Patch v3 22/22] Readme.md: Add CI build status badges Thread-Topic: [Patch v3 22/22] Readme.md: Add CI build status badges Thread-Index: AQHVjs3m0HC+g86FtEKK3MjUG81INad0LNmw Date: Thu, 31 Oct 2019 04:42:36 +0000 Message-ID: References: <20191029195517.20028-1-michael.d.kinney@intel.com> <20191029195517.20028-23-michael.d.kinney@intel.com> <20191030025810.GV16820@bivouac.eciton.net> In-Reply-To: <20191030025810.GV16820@bivouac.eciton.net> 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.138] 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 Leif, Thanks for noticing the reference to the ci dir. That content Has been moved to .azurepipelines directory and the Readme.md=20 filename is correct there. I will split out the format changes to this Readme.md into its own patch so it will not be mixed with the CI features. I agree that the table with CI status is not very readable. This is true in MD files for most long hyperlinks, especially when they are put into a table format. We can see if we can improve the readability to make it easier to maintain, but=20 will likely still be long lines. Mike > -----Original Message----- > From: Leif Lindholm > Sent: Tuesday, October 29, 2019 7:58 PM > To: Kinney, Michael D > Cc: devel@edk2.groups.io; Andrew Fish ; > Laszlo Ersek > Subject: Re: [Patch v3 22/22] Readme.md: Add CI build > status badges >=20 > On Tue, Oct 29, 2019 at 12:55:17PM -0700, Michael D > Kinney wrote: > > https://bugzilla.tianocore.org/show_bug.cgi?id=3D2315 > > > > Update Readme.md with status badges from the most > recent EDK II > > Continuous Integration(CI) build. > > > > Cc: Andrew Fish > > Cc: Laszlo Ersek > > Cc: Leif Lindholm > > Signed-off-by: Michael D Kinney > > > --- > > Readme.md | 29 ++++++++++++++++++++++------- > > 1 file changed, 22 insertions(+), 7 deletions(-) > > > > diff --git a/Readme.md b/Readme.md > > index 27e4ce0771..37183c3265 100644 > > --- a/Readme.md > > +++ b/Readme.md > > @@ -3,6 +3,19 @@ > > A modern, feature-rich, cross-platform firmware > development > > environment for the UEFI and PI specifications from > www.uefi.org. > > > > +## Build Status > > + > > +| Host Type | Toolchain | Branch | Build Status | > Test Status | Code > > +| Coverage | >=20 > This may sound like nitpicking, but I think it's > important for > messaging: we aren't listing tolchains, we are listing > toolchain profiles. For VS, we are actually adding new > profiles for each version, so that statement is not > misleading. However, GCC5 is highly misleading (it is > the profile we use for anything gcc5 onwards). >=20 > I don't think we can usefully pull in information about > the toolchain actually used, so I think the name of the > column should really be toolchain profile. >=20 > Even given that, I think it is time we create an > abstracted GCC_CURRENT toolchain profile, pointing to > GCC5. But if we rename the column, that may not be a > prerequisite for this set. >=20 > > +| :-------- | :-------- | :---- | :----- | :---- | :- > -- | Windows | > > +| VS2019 | master | [![Build > > +| Status](https://dev.azure.com/tianocore/edk2-ci- > play/_apis/build/st > > +| > atus/Edk2%20Windows%20VS2019%20CI?branchName=3Dmaster)](ht > tps://dev.a > > +| zure.com/tianocore/edk2-ci- > play/_build/latest?definitionId=3D14&branc > > +| hName=3Dmaster) | [![Azure DevOps > > +| tests](https://img.shields.io/azure- > devops/tests/tianocore/edk2-ci- > > +| play/14.svg)](https://dev.azure.com/tianocore/edk2- > ci-play/_build/l > > +| atest?definitionId=3D14&branchName=3Dmaster) | [![Code > > +| Coverage](https://img.shields.io/badge/coverage- > coming_soon-blue)]( > > +| https://dev.azure.com/tianocore/edk2-ci- > play/_build/latest?definiti > > +| onId=3D14&branchName=3Dmaster)| Ubuntu | GCC5 | master > | [![Build > > +| Status](https://dev.azure.com/tianocore/edk2-ci- > play/_apis/build/st > > +| > atus/Edk2%20Ubuntu%20GCC5%20CI?branchName=3Dmaster)](https > ://dev.azur > > +| e.com/tianocore/edk2-ci- > play/_build/latest?definitionId=3D15&branchNa > > +| me=3Dmaster) | [![Azure DevOps > > +| tests](https://img.shields.io/azure- > devops/tests/tianocore/edk2-ci- > > +| play/15.svg)](https://dev.azure.com/tianocore/edk2- > ci-play/_build/l > > +| atest?definitionId=3D15&branchName=3Dmaster) | [![Code > > +| Coverage](https://img.shields.io/badge/coverage- > coming_soon-blue)]( > > +| https://dev.azure.com/tianocore/edk2-ci- > play/_build/latest?definiti > > +| onId=3D15&branchName=3Dmaster) | >=20 > I have one problem with the above, which is that it > turns an otherwise very readable in text mode file > (which is half the point of .md) into what looks a > little bit like a transmission error. >=20 > While I agree there is value in having this on the > default loaded github webpage, do we have other options? >=20 > > + > > +[More CI Build information](ci/readme.md) >=20 > I don't see this file in the tree? > Which is good, beacuse it doesn't follow name standards. >=20 > > + > > +## License Details > > + > > +[![License](https://img.shields.io/badge/license-BSD- > -2--Clause--Pate > > +nt-blue)](License.txt) > > + >=20 > I don't mind this addition, but I think it should be a > separate patch. > I also think it would be best to hold this back until > we've submoduled some of those otherwise licensed > components. > There is a very real likelihood of people skimming, > seeing the image and ignoring the text. >=20 > > The majority of the content in the EDK II open source > project uses a > > [BSD-2-Clause Plus Patent License](License.txt). The > EDK II open > > source project contains the following components that > are covered by additional licenses: > > @@ -20,7 +33,8 @@ contains the following components > that are covered by additional licenses: > > The EDK II Project is composed of packages. The > maintainers for each > > package are listed in > [Maintainers.txt](Maintainers.txt). > > > > -# Resources > > +## Resources > > + >=20 > No objections to change, but it is unrelated to this > patch. >=20 > > * [TianoCore](http://www.tianocore.org) > > * [EDK > > > II](https://github.com/tianocore/tianocore.github.io/wik > i/EDK-II) > > * [Getting Started with EDK > > > II](https://github.com/tianocore/tianocore.github.io/wik > i/Getting-Star > > ted-with-EDK-II) @@ -29,7 +43,8 @@ are listed in > > [Maintainers.txt](Maintainers.txt). > > * [How To > > > Contribute](https://github.com/tianocore/tianocore.githu > b.io/wiki/How- > > To-Contribute) > > * [Release > > > Planning](https://github.com/tianocore/tianocore.github. > io/wiki/EDK-II > > -Release-Planning) > > > > -# Code Contributions > > +## Code Contributions > > + >=20 > No objection to change, but it is unrelated to this > patch. >=20 > > To make a contribution to a TianoCore project, follow > these steps. > > 1. Create a change description in the format > specified below to > > use in the source control commit log. > > @@ -57,7 +72,7 @@ To make a contribution to a > TianoCore project, follow these steps. > > Contributions using other licenses might be > accepted, but further > > review will be required. > > > > -# Developer Certificate of Origin > > +## Developer Certificate of Origin >=20 > No objection to change, but it is unrelated to this > patch. >=20 > > > > Your change description should use the standard > format for a commit > > message, and must include your `Signed-off-by` > signature. > > @@ -104,7 +119,7 @@ By making a contribution to this > project, I certify that: > > this project or the open source license(s) > involved. > > ``` > > > > -# Sample Change Description / Commit Message > > +## Sample Change Description / Commit Message >=20 > No objection to change, but it is unrelated to this > patch. >=20 > > > > ``` > > From: Contributor Name @@ - > 115,14 +130,14 > > @@ Full-commit-message > > Signed-off-by: Contributor Name > ``` > > > > -## Notes for sample patch email > > +### Notes for sample patch email >=20 > No objection to change, but it is unrelated to this > patch. >=20 > > > > * The first line of commit message is taken from the > email's subject > > line following `[Repository/Branch PATCH]`. The > remaining portion of the > > commit message is the email's content. > > * `git format-patch` is one way to create this format > > > > -## Definitions for sample patch email > > +### Definitions for sample patch email >=20 > No objection to change, but it is unrelated to this > patch. >=20 > > > > * `Repository` is the identifier of the repository > the patch applies. > > This identifier should only be provided for > repositories other than > > @@ -141,7 +156,7 @@ Signed-off-by: Contributor Name > > > > * `Signed-off-by` is the contributor's signature > identifying them > > by their real/legal name and their email address. > > > > -# Submodules > > +## Submodules >=20 > No objection to change, but it is unrelated to this > patch. >=20 > / > Leif >=20 > > > > Submodule in EDK II is allowed but submodule chain > should be avoided > > as possible as we can. Currently EDK II contains two > submodules > > -- > > 2.21.0.windows.1 > >