Skip to content
This repository has been archived by the owner on Nov 28, 2019. It is now read-only.

WIP: website #75

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

WIP: website #75

wants to merge 4 commits into from

Conversation

ceball
Copy link
Member

@ceball ceball commented Nov 29, 2017

Site should appear at https://ioam.github.io/paramnb/ (untested)

To do:

  • write to-do list

@jbednar
Copy link
Member

jbednar commented Nov 29, 2017

Should we merge it just to test whether a site appears?

@ceball
Copy link
Member Author

ceball commented Nov 29, 2017

The travis config is to build the site for changes to the docs2 branch (i.e. this branch), so a site has already appeared. Same for all the website PRs I've opened recently: the sites are rebuilt for pushes to the docs branch. When the site is basically done, and ready to merge, the travis config should be adjusted to deploy the site only on tag or similar kind of a thing.

Unfortunately something seems to have gone very wrong with the paramnb site - the notebooks haven't run at al...

@jbednar
Copy link
Member

jbednar commented Nov 29, 2017

The build policy sounds good. In the long run, PRs should probably deploy to a "build" or "test" url, and then an appropriate tag on master can deploy to the main site.

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

Successfully merging this pull request may close these issues.

2 participants