From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f43.google.com (mail-wm1-f43.google.com [209.85.128.43]) by mx.groups.io with SMTP id smtpd.web11.2231.1573168606352168190 for ; Thu, 07 Nov 2019 15:16:46 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@linaro.org header.s=google header.b=e/sY+kbA; spf=pass (domain: linaro.org, ip: 209.85.128.43, mailfrom: leif.lindholm@linaro.org) Received: by mail-wm1-f43.google.com with SMTP id l17so3517971wmh.0 for ; Thu, 07 Nov 2019 15:16:46 -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=uylb9zsYaP2coVUrz5MzWyH0V6h6wzsineVTcyX83yo=; b=e/sY+kbAyzEV/SLntbhT/mv4VNoy3XwVdfIiSS8DYSiFpJBfpUg3XYnrpitGOFqvuH R+Hj7nu3E7aeY61iEnwnCa7yjkjuV+/7bofdkUDu1pAUefikVRU6vzhi+BOgcOqm93oM ersIoNUyC29JQq4dghe3nRrN/j3UC+AkatFWQsVzlbQECbICwwdn9CN0FZgZxIbK4BPs lyNPWxmAVg9AIZ7Kp7jeRn1TasLG+MUDPNv0nAYqocddlft3ga5pxDtwQQFvPztSukxk s6eAxCVl38KMRShdptIKhR7uKA4kA1EzwH7VHbx9/IubsdmE7oiM8YH0gawKSMPXBnuq Ou2g== 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=uylb9zsYaP2coVUrz5MzWyH0V6h6wzsineVTcyX83yo=; b=lkv55WtYlsTnwjcIimBvVj8jszT6FKiAmpkjTjCziLImo4Cs6ye7Z8+6nakDZ98z7R QDbNR6U9ZrEnoSiqjdHwUrDQBILjFy+nbs3lnuXPhHF1X3iVRyepdO3UXSA5TCQfU45t /CR/XQ7NnGD+pmQbKPAZ5IvRUDsC+iLg13XpdWih6fYESM7I9WJW/FB6fUHfP5o/ScL9 YQqdsgzmmKcZMbkzY+612khSSsz9KnX+8LNwxSQ5XhlfPp51ONz6HNtBVGsBjoVFvgmG 2q0DXftNmgsawFNjOTObvGl/Etnc++xJFGODr44U3quySoO8hhoZ1EI0VYcPHTNao0uV 6dDg== X-Gm-Message-State: APjAAAVGTUZghA6ki9ViMFBKRgNBfN9HTf1fHou1sCnpk5I7IQwjTl9k ilKmxZ+LlwaLaQiuRapg2Jf5Zw== X-Google-Smtp-Source: APXvYqwp/1mUtxRd5XSEYaUi6w023/snC393agW/h2e/bQfyzd7mJ1JfHDSO/XCd3nv+UuhInIZ4sA== X-Received: by 2002:a7b:c642:: with SMTP id q2mr5342622wmk.169.1573168604879; Thu, 07 Nov 2019 15:16:44 -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 t5sm3499076wro.76.2019.11.07.15.16.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 07 Nov 2019 15:16:43 -0800 (PST) Date: Thu, 7 Nov 2019 23:16:41 +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: <20191107231641.GA16820@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 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. As do I. Just not at any cost (including nullifying the entire point of markdown). > 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. I think that might work. > 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? My main problem is that the current version basically looks like a transfer error. If we can make it look like a compartmentalised section of the file that provides real value when viewed through another tool, I'm quite OK with that. 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 > > >