From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f67.google.com (mail-wr1-f67.google.com [209.85.221.67]) by mx.groups.io with SMTP id smtpd.web10.7740.1573205067868761201 for ; Fri, 08 Nov 2019 01:24:28 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@linaro.org header.s=google header.b=IuMKT8aU; spf=pass (domain: linaro.org, ip: 209.85.221.67, mailfrom: leif.lindholm@linaro.org) Received: by mail-wr1-f67.google.com with SMTP id f2so6122929wrs.11 for ; Fri, 08 Nov 2019 01:24:27 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=oXi1A2TDt3z8QgG9xpuOVFqG5J40TJHW3nrC5MhIVSA=; b=IuMKT8aUK36KiZ7T7tW1U0wC1dMeEdHbu14X9VRGGcqS63NMpozX+XT8+JoN20DXaY IMhTjbYfW0WEK6Sy706zQ01Xb3ThfGG9jaAuZUCdZXWwcLejTb/RSN6EV7ETs/6DJi87 hGjL3qvDCVrAg8mWyFTFNZZuKbXGxf8dtZ8OOet0JV2DX90AH3xlrWUcHEZ3XPKebdjm ewir6XL3mHbAO2oyyEEpswYCZLyv05PuCt4kyPnVGUxV38D9ouduXpbC27M8nSw8Ozwx rH/mpCQN9PztsYmkQ7kJ+0hWLcGS1MO3+0b+EdNB9RC2lgmdvRc9uUepJv+ZqRjS5oTo oGBg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=oXi1A2TDt3z8QgG9xpuOVFqG5J40TJHW3nrC5MhIVSA=; b=Z9RnSReIokIYdEHGU6wc6s+us0j0atRph5XavjAgh/bAvpRcKcAoKqvu98XpijDM9A iudkjOVgwbpdzvWLBBhrTCpU4NlhrFNHKMw3+aq/KMcyuwfDr9yaWk46n4r29ePEPqCY UqYdpFOSREXplQtVq4uhFPyuEXsuzwHodr/84ciznjWA9avWBM7wiUdugSrjMt/F0oWE 4TGF8xYOIde08C9yYRnV8uXT4hGkFWqsAZvkqYDjw+04nGcidtatLQoasan8gU+YDLlN jtNMvST5R10CDKVeAsZ+8fvzWHEHEv03YfeHPTAp2gVHAn+VYDfXOdQTwZ1l8MLiP6pI Q1uA== X-Gm-Message-State: APjAAAXi4Apq7jQuoMNgylQV4hzbRglogHqeP2rLQx1lT4qwyDtkWQgQ 2XU9dUeiTNcLaxe60VvKS3rxXg== X-Google-Smtp-Source: APXvYqw2+Gc+zgbcJSVNt1zLGaYXyyBILYIXmHFl2wqUiDa+emzId0shfNIAjT1tax4b4pyU65WZeQ== X-Received: by 2002:adf:e5c4:: with SMTP id a4mr7667955wrn.334.1573205066246; Fri, 08 Nov 2019 01:24:26 -0800 (PST) Return-Path: Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id 76sm6562710wma.0.2019.11.08.01.24.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 08 Nov 2019 01:24:25 -0800 (PST) Date: Fri, 8 Nov 2019 09:24:23 +0000 From: "Leif Lindholm" To: "Kinney, Michael D" Cc: Sean Brogan , "devel@edk2.groups.io" , Andrew Fish , Laszlo Ersek Subject: Re: [Patch v4 22/22] Readme.md: Add CI build status badges Message-ID: <20191108092423.GB16820@bivouac.eciton.net> References: <20191107011349.16524-1-michael.d.kinney@intel.com> <20191107011349.16524-23-michael.d.kinney@intel.com> <20191107160029.GN16820@bivouac.eciton.net> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi Mike, On Thu, Nov 07, 2019 at 07:42:27PM +0000, Kinney, Michael D wrote: > 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 > 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? I think that would be a lot better - the crazy line lengths is the biggest issue. Best Regards, Leif > 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 > > > > 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=master)](https: > > //dev.azure.com/tianocore/edk2-ci- > > play/_build/latest?definitionId=27&branchName=master) | > > [![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=27&branchName=master) | > > [![Code Coverage](https://img.shields.io/badge/coverage- > > coming_soon-blue)](https://dev.azure.com/tianocore/edk2- > > ci-play/_build/latest?definitionId=27&branchName=master) > > | > > > +| Ubuntu | GCC | master | [![Build > > Status](https://dev.azure.com/tianocore/edk2-ci- > > play/_apis/build/status/edk2- > > staging/Ubuntu%20GCC5%20CI?branchName=master)](https://d > > ev.azure.com/tianocore/edk2-ci- > > play/_build/latest?definitionId=26&branchName=master) > > | [![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=26&branchName=master) | > > [![Code Coverage](https://img.shields.io/badge/coverage- > > coming_soon-blue)](https://dev.azure.com/tianocore/edk2- > > ci-play/_build/latest?definitionId=26&branchName=master) > > | > > > > 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? > > > > / > > Leif. > > > > > + > > > +[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 > > >