- From: Raymond Toy <rtoy@google.com>
- Date: Tue, 20 Mar 2018 23:01:55 +0000
- To: Paul Adenot <padenot@mozilla.com>
- Cc: Tab Atkins <tabatkins@google.com>, "public-audio@w3.org Group" <public-audio@w3.org>
- Message-ID: <CAE3TgXFTwzjzY3018jUAvw5W_L2GBxBaP5RD6P3w69NECkTTww@mail.gmail.com>
I've updated Travis CI so that it will at least tell you if there are new errors. This is kind of fragile because I had to check in the current list of errors. CI will just run bikeshed and compare the errors from that with the expected list. If there are no differences, you're good to go. Have not yet integrated construction of the actual html file automatically. I think that will take some time to do. On Tue, Mar 20, 2018 at 2:36 PM Raymond Toy <rtoy@google.com> wrote: > Oh. Because our spec includes some bikeshed macro files that are separate > from the index.bs file, the remote usage of bikeshed will be missing some > things. > > > On Tue, Mar 20, 2018 at 1:34 PM Raymond Toy <rtoy@google.com> wrote: > >> PR preview should be working now. It's a bit slow, so be patient. The >> "Preview | Diff" button may take a while before showing up. >> >> >> >> On Tue, Mar 20, 2018 at 10:36 AM Raymond Toy <rtoy@google.com> wrote: >> >>> I've added https://github.com/WebAudio/web-audio-api/wiki/UsingBikeshed >>> with notes to make it easier to find this information. Also added the >>> current output so you know what issues are currently in the bikeshed >>> version of the spec. >>> >>> When you make changes, be sure to check the output against the list to >>> make sure you haven't introduced new warnings or errors. Sorry it's so >>> manual right now. >>> >>> I'll look into getting the preview working and also getting travis >>> working again (that might be a bit difficult since we have warnings and >>> issues in the current spec). >>> >>> >>> On Tue, Mar 13, 2018 at 3:08 AM Paul Adenot <padenot@mozilla.com> wrote: >>> >>>> Thanks Tab for bikeshed and all the help in doing the conversion, and >>>> also for those infos. >>>> >>>> We've had pr-preview in place for use with ReSpec, it's indeed quite a >>>> nice tool. Hopefully it still works with our bikeshed setup, I hear we have >>>> a couple errors still. >>>> >>>> Paul. >>>> >>>> >>>> >>>> On Mon, Mar 12, 2018 at 10:31 PM, Tab Atkins <tabatkins@google.com> >>>> wrote: >>>> >>>>> On Thu, Mar 8, 2018 at 1:21 PM Raymond Toy <rtoy@google.com> wrote: >>>>> >>>>>> The bikeshed version has been committed. (See >>>>>> https://github.com/WebAudio/web-audio-api/pull/1514) >>>>>> >>>>>> Travis CI is basically ineffective now because there are errors with >>>>>> the current spec (some due to bokeshed). >>>>>> >>>>>> So you'll have to check manually to make sure an PRs work. This also >>>>>> means you need a copy of bikeshed locally (probably). See >>>>>> https://tabatkins.github.io/bikeshed/ for details on how to use >>>>>> bikeshed. >>>>>> >>>>> >>>>> No need to have it locally (tho that does make some things more >>>>> convenient). You can always use the network version, either via curl or an >>>>> HTML form, as documented at < >>>>> https://tabatkins.github.io/bikeshed/#remote>. >>>>> >>>>> >>>>>> We will want to set up travis ci to generate the html version >>>>>> automatically. Hopefully soon. >>>>>> >>>>> >>>>> In addition to Travis automatically handling the build for you, you >>>>> can set up PR Preview <https://github.com/tobie/pr-preview> to >>>>> auto-generate HTML versions from PRs. >>>>> >>>>> >>>>>> And finally, many thanks to Tab Atkins who volunteered to do the >>>>>> conversion for us and did practically the entire spec by himself. I just >>>>>> cleaned up a few issues that would have been difficult for someone >>>>>> unfamiliar with the spec to know about. Any problems with this version are >>>>>> all due to me. >>>>>> >>>>> >>>>
Received on Tuesday, 20 March 2018 23:02:29 UTC