ngff icon indicating copy to clipboard operation
ngff copied to clipboard

PR rendering CI

Open ivirshup opened this issue 1 year ago • 6 comments

The links for rendered versions of the PRs haven't been working for me for at least the last week. Does anyone know what's up with this?

I'm also wondering if this whole process could be improved by building the html and deploying it somewhere ( like readthedocs PR builds). Even when the links were working they did take a while to load. It would also be nice to see build errors and warnings from CI.

ivirshup avatar Oct 09 '22 13:10 ivirshup

The W3 service that generates the bikeshed-based pages has definitely become unreliable.

Additionally, I've searched for solutions for PR previews of GitHub Pages but to no avail. The closest "recommendation" I've seen to date is to create a repository per PR.

So :+1: from me.

joshmoore avatar Oct 09 '22 14:10 joshmoore

PR previews of GitHub Pages

Yeah, this doesn't seem straight forward. Maybe we could just use readthedocs?

I think we can also do some partial fixes easily (like, at least checking that the doc builds) even if we can't view it super easily.


Some workarounds for viewing the rendered spec

  • Rendering by pasting the index.bs url here: https://api.csswg.org/bikeshed/ works
  • pipx install bikeshed, bike shed update, and bikeshed spec from the version root directory (e.g. latest, 0.4) will generate a rendered version
    • bikeshed pins its dependencies to specific versions, so don't install it into an environment with anything else

More info on running bike shed (including some CI actions) here: https://tabatkins.github.io/bikeshed/

ivirshup avatar Oct 09 '22 15:10 ivirshup

@joshmoore, I'm not sure it's the w3c service that's preventing rendering at the moment. I think it's the github action.

The links in the tables PR comment looks like:

http://api.csswg.org/bikeshed/?url=https://raw.githubusercontent.com/ome/ngff//latest/index.bs

When it should look like:

http://api.csswg.org/bikeshed/?url=https://github.com/ome/ngff/raw/cc83a82c716670fb60d2d7f8a89f4f700a17b788/latest/index.bs

The action doesn't seem to be using the environment variable properly. You can also see this in the action logs.

ivirshup avatar Oct 09 '22 17:10 ivirshup

@joshmoore, I suspect rtd may be a bit of a pain here because we would have to work around sphinx.

But a colleague has recently used netlify to host their PR builds (see https://github.com/scverse/scverse.github.io/issues/63), which I found fairly straight forward to set up. See an example here: ivirshup/ngff#1

ivirshup avatar Oct 11 '22 15:10 ivirshup