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? Thanks, Mike > -----Original Message----- > From: Leif Lindholm <leif.lindh...@linaro.org> > Sent: Thursday, November 7, 2019 8:00 AM > To: Kinney, Michael D <michael.d.kin...@intel.com> > Cc: devel@edk2.groups.io; Andrew Fish <af...@apple.com>; > Laszlo Ersek <ler...@redhat.com> > 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 <af...@apple.com> > > Cc: Laszlo Ersek <ler...@redhat.com> > > Cc: Leif Lindholm <leif.lindh...@linaro.org> > > Signed-off-by: Michael D Kinney > <michael.d.kin...@intel.com> > > --- > > 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 | [](https: > //dev.azure.com/tianocore/edk2-ci- > play/_build/latest?definitionId=27&branchName=master) | > [](https://dev.azure.com/tianocore/edk2-ci- > play/_build/latest?definitionId=27&branchName=master) | > [](https://dev.azure.com/tianocore/edk2- > ci-play/_build/latest?definitionId=27&branchName=master) > | > > +| Ubuntu | GCC | master | [](https://d > ev.azure.com/tianocore/edk2-ci- > play/_build/latest?definitionId=26&branchName=master) > | [](https://dev.azure.com/tianocore/edk2-ci- > play/_build/latest?definitionId=26&branchName=master) | > [](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 > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#50262): https://edk2.groups.io/g/devel/message/50262 Mute This Topic: https://groups.io/mt/44874114/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
