From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by mx.groups.io with SMTP id smtpd.web10.9585.1573155749404784458 for ; Thu, 07 Nov 2019 11:42:29 -0800 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.136, mailfrom: michael.d.kinney@intel.com) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 Nov 2019 11:42:28 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.68,279,1569308400"; d="scan'208";a="213087455" Received: from orsmsx106.amr.corp.intel.com ([10.22.225.133]) by fmsmga001.fm.intel.com with ESMTP; 07 Nov 2019 11:42:28 -0800 Received: from orsmsx152.amr.corp.intel.com (10.22.226.39) by ORSMSX106.amr.corp.intel.com (10.22.225.133) with Microsoft SMTP Server (TLS) id 14.3.439.0; Thu, 7 Nov 2019 11:42:28 -0800 Received: from orsmsx113.amr.corp.intel.com ([169.254.9.28]) by ORSMSX152.amr.corp.intel.com ([169.254.8.219]) with mapi id 14.03.0439.000; Thu, 7 Nov 2019 11:42:28 -0800 From: "Michael D Kinney" To: Leif Lindholm , "Kinney, Michael D" , Sean Brogan CC: "devel@edk2.groups.io" , Andrew Fish , Laszlo Ersek Subject: Re: [Patch v4 22/22] Readme.md: Add CI build status badges Thread-Topic: [Patch v4 22/22] Readme.md: Add CI build status badges Thread-Index: AQHVlYSHWxJP6KMsakSZ0rAy5D9NQqeAGdhg Date: Thu, 7 Nov 2019 19:42:27 +0000 Message-ID: References: <20191107011349.16524-1-michael.d.kinney@intel.com> <20191107011349.16524-23-michael.d.kinney@intel.com> <20191107160029.GN16820@bivouac.eciton.net> In-Reply-To: <20191107160029.GN16820@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.139] 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, I really like the build status information in Readme.md so it is visible whenever anyone navigates to the edk2 repo. This matches what many other open source projects provide. GitHub markdown does support HTML, so one idea is to=20 put this content into an HTML table and that would allow us to format the Readme.md file with each hyperlink on its own line. If we remove the content from a markdown table, then there are some other pure markdown methods to put each hyperlink on its own line. If we can get down to one hyperlink per line in Readme.md, does that address your concerns?=20 Thanks, Mike > -----Original Message----- > From: Leif Lindholm > Sent: Thursday, November 7, 2019 8:00 AM > To: Kinney, Michael D > Cc: devel@edk2.groups.io; Andrew Fish ; > Laszlo Ersek > Subject: Re: [Patch v4 22/22] Readme.md: Add CI build > status badges >=20 > On Wed, Nov 06, 2019 at 05:13:49PM -0800, Michael D > Kinney wrote: > > 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 | 11 +++++++++++ > > 1 file changed, 11 insertions(+) > > > > diff --git a/Readme.md b/Readme.md > > index 27e4ce0771..0207923763 100644 > > --- a/Readme.md > > +++ b/Readme.md > > @@ -3,6 +3,17 @@ > > 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 | > > +| :-------- | :-------- | :------ | :----------- | :- > --------- | :------------ | > > +| Windows | VS2019 | master | [![Build > Status](https://dev.azure.com/tianocore/edk2-ci- > play/_apis/build/status/edk2- > staging/Windows%20VS2019%20CI?branchName=3Dmaster)](https: > //dev.azure.com/tianocore/edk2-ci- > play/_build/latest?definitionId=3D27&branchName=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/latest?definitionId=3D27&branchName=3Dmaster) | > [![Code Coverage](https://img.shields.io/badge/coverage- > coming_soon-blue)](https://dev.azure.com/tianocore/edk2- > ci-play/_build/latest?definitionId=3D27&branchName=3Dmaster) > | > > +| Ubuntu | GCC | master | [![Build > Status](https://dev.azure.com/tianocore/edk2-ci- > play/_apis/build/status/edk2- > staging/Ubuntu%20GCC5%20CI?branchName=3Dmaster)](https://d > ev.azure.com/tianocore/edk2-ci- > play/_build/latest?definitionId=3D26&branchName=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/latest?definitionId=3D26&branchName=3Dmaster) | > [![Code Coverage](https://img.shields.io/badge/coverage- > coming_soon-blue)](https://dev.azure.com/tianocore/edk2- > ci-play/_build/latest?definitionId=3D26&branchName=3Dmaster) > | >=20 > I'm still not super happy about this. > If there is no hacky way of embedding this from another > file, could we consider breaking it out into a separate > BuildStatus.md and putting a link to that at the top of > this file? >=20 > / > Leif. >=20 > > + > > +[More CI Build information](.pytool/Readme.md) > > + > > +# License Details > > + > > 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: > > -- > > 2.21.0.windows.1 > >