Skip to content
This repository has been archived by the owner on Feb 16, 2023. It is now read-only.

Autogenerate index.html #59

Open
marcoscaceres opened this issue Feb 5, 2018 · 15 comments
Open

Autogenerate index.html #59

marcoscaceres opened this issue Feb 5, 2018 · 15 comments
Assignees

Comments

@marcoscaceres
Copy link
Contributor

We need to find a way of having TravisCI autogenerating the spec. We shouldn't be seeing the generated version as part of a pull request. If generation fails, then it means something is wrong with the spec.

Maybe there is some guidance over in the BikeShed repo about this? Let's ping Tab if we don't find anything there.

@beep
Copy link
Collaborator

beep commented Feb 8, 2018

@marcoscaceres Sounds great, thank you.

@hereinthehive @tomhodgins, assigning this to you to investigate.

@tomhodgins
Copy link
Collaborator

I'm not sure how to set this up to autobuild on commit in Github. Would that mean we would also be ignoring the HTML file so we could rebuild our own HTML as we edit, and then commit only the BS file to the repository?

Editing a Bikeshed document means rebuilding it a lot while working on it to try to get rid of those error messages, it can be a little picky at times.

Does anybody have more general experience setting up some kind of a build step after commit on Github we could adapt to Bikeshed?

@hereinthehive
Copy link
Collaborator

@tomhodgins I've only ever done this using NPM scripts...fairly sure I have!

@marcoscaceres
Copy link
Contributor Author

Hey, so I poked around some of the other repos and found one that solves it:

https://github.com/WICG/picture-in-picture

We need to steal their bash script and travis file (and I guess encrypt credentials so that git merge and push can happen back to the gh-pages branch). Effectively, we would go back to using a "master" branch, and auto-publish to the gh-pages branch.

Anyone feeling brave want to take this on?

@tabatkins
Copy link
Contributor

At-mention me if you want me to pay attention to a thread. ^_^

Yeah, @domenic has a good auto-deploy-from-Travis script setup for the WHATWG specs; I haven't copied it into the Bikeshed docs yet because I keep forgetting where it is. Maybe he can enlighten us. ^_^

@domenic
Copy link

domenic commented Feb 9, 2018

https://github.com/whatwg/whatwg.org/blob/master/resources.whatwg.org/build/deploy.sh is our very comprehensive WHATWG spec build script, including e.g. HTML checking, linting, checking for errors/warnings before running the actual thing, generating commit snapshots separately from living standards, generating a service worker, ... Notably it doesn't upload to gh-pages.

Probably good enough for most purposes is my Travis-CI focused gist: https://gist.github.com/domenic/ec8b0fc8ab45f39403dd

@domenic
Copy link

domenic commented Feb 9, 2018

Although https://docs.travis-ci.com/user/deployment/pages/ seems new and perhaps much easier.

@marcoscaceres
Copy link
Contributor Author

@domenic, thanks heaps! @tabatkins, we didn't want to bother you unless we got really stuck.

In any case, working out a generalized solution for this would be quite helpful (and adding that somewhere to BS's docs).. sounds like we are close to that with @domenic's script.

@marcoscaceres
Copy link
Contributor Author

So using deploy, looks like we would just run BikeShed as one would locally. That should be fairly straight forward. We probably don't need any additional scripts unless we want to do additional linting, etc., which I don't think we need for this doc.

@beep
Copy link
Collaborator

beep commented Feb 12, 2018

@tomhodgins @hereinthehive Does #59 (comment) provide a bit of guidance/assistance?

@eeeps
Copy link
Collaborator

eeeps commented Mar 30, 2018

Any progress on this, anybody? I was surprised to see today that #64 wasn’t reflected in https://wicg.github.io/cq-usecases/

@beep
Copy link
Collaborator

beep commented Jul 23, 2018

@tomhodgins @hereinthehive Have either of you gotten a chance to work on this?

@eeeps
Copy link
Collaborator

eeeps commented Aug 7, 2018

Possibly helpful: WICG/resize-observer#60

@tomhodgins
Copy link
Collaborator

Ah thanks! I just got a new computer yesterday so I can look into this stuff again now :D

@eeeps
Copy link
Collaborator

eeeps commented Oct 1, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants