w3c / media-source

Media Source Extensions
https://w3c.github.io/media-source/
Other
268 stars 58 forks source link

Automate publication to /TR and handle static files #303

Closed tidoust closed 2 years ago

tidoust commented 3 years ago

This update completes the job that validates and publishes the spec to handle static files as well (images, companion documents, MSE v1 errata) and to now publish Working Draft updates automatically to /TR.

The ECHIDNA_TOKEN secret was added to the repo settings.

wolenetz commented 2 years ago

@tidoust Looking at previous, similar https://github.com/w3c/picture-in-picture/pull/200, especially the further pieces added to that to fix that spec's automatic publication, does this current change also need:

wolenetz commented 2 years ago

Hmm. I'm also uncertain how to unmark approval for this until I'm clear that my questions in https://github.com/w3c/media-source/pull/303#issuecomment-1042467586 are answered. For now, I've just self-requested a review, but really I'm pending @tidoust responding.

tidoust commented 2 years ago

Looking at previous, similar https://github.com/w3c/picture-in-picture/pull/200

The Picture-in-Picture spec uses Bikeshed. MSE uses ReSpec. ReSpec should not need a previous link and the shortname in the ReSpec config is already set to media-source-2. We didn't reflect the notion of level in the title in the First Public Working Draft, so no need to go beyond that (if that's something that feels needed, ReSpec's level config parameter could be used but that is orthogonal to automation).

I'm afraid the only real way to test the publication workflow is to put it in production, so I'll take the liberty to go ahead and merge this PR. I'll fix parameters if publication fails.

tidoust commented 2 years ago

There remained a parameter name hiccup in the job. Now fixed. I confirm that automatic publication is now up and running: https://www.w3.org/TR/2022/WD-media-source-2-20220217/

wolenetz commented 2 years ago

Ack. Thank you!