W3C home > Mailing lists > Public > public-test-infra@w3.org > October to December 2016

Re: Verbosity ot wpt-stability-bot

From: James Graham <james@hoppipolla.co.uk>
Date: Thu, 1 Dec 2016 21:49:06 +0000
To: public-test-infra@w3.org
Message-ID: <24a43d5d-0170-13ed-2a68-04a8f547200c@hoppipolla.co.uk>
On 30/11/16 19:01, Philip J├Ągenstedt wrote:
> Hi all,
>
> I asked James about this and he asked me to seek feedback here.
>
> The wpt-stability-bot is amazing, but I wonder if there are any options for
> making it generate less email. On
> https://github.com/w3c/web-platform-tests/pull/4250 I did a lot of fixups
> while experimenting, and wpt-stability-bot has posted a total of 18
> comments, obscuring any real discussion.
>
> I think seeing the pass/fail results is useful even if it's stable, so some
> visibility is great. How about it the bot only added one comment, and kept
> updating that with the most recent info?

Done.

Note that using the details element for the results, as has also been 
suggested, is non-trivial, because:

  * The GitHub stylesheet breaks the disclosure triangle in 
spec-conformant browsers
  * It isn't possible to add inline style attributes
  * It isn't possible to have markdown inside an explicit tag, so the 
tables would need to be rewritten using explicit <table>+etc. elements
  * The current code logs exactly the same plaintext to the travis log 
and the github comment, using the fact that markdown is more or less 
readable as text. Therefore replacing all the markdown with explicit 
tags is highly undesirable since human readability of the logs is 
essential when PRs are not submitted to w3c/web-platform-tests but a fork.
Received on Thursday, 1 December 2016 21:49:35 UTC

This archive was generated by hypermail 2.4.0 : Friday, 17 January 2020 17:34:12 UTC