Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation deployment inconsistent with package deployment #1231

Closed
2 tasks
shilman opened this issue Jun 9, 2017 · 7 comments
Closed
2 tasks

Documentation deployment inconsistent with package deployment #1231

shilman opened this issue Jun 9, 2017 · 7 comments

Comments

@shilman
Copy link
Member

shilman commented Jun 9, 2017

why

  • Documentation goes live as soon as its merged to master
  • Packages go live as soon as somebody manually releases them (@shilman has been doing it about once a week so far)

This causes the documentation to be temporarily out of sync, and problems like this which is bad for users: #1230

what

Either:

  • Make documentation fixes part of the release process (somehow)
  • Automate releases so that they happen as soon as a PR is merged
@tmeasday
Copy link
Member

Unless we want to go all CD, I would recommend a vNEXT branch on the docs that is the "base" branch for PRs etc, and merge to master on release.

If we want to get really fancy we could do what we did for Meteor and have a deploy per release and a vNEXT deploy (http://docs.meteor.com/vNEXT/) but that's probably way over the top ;)

@shilman
Copy link
Member Author

shilman commented Jun 10, 2017

@tmeasday Chatted with @ndelangen in Slack and he proposed that we only deploy docs alongside lerna publish. This makes sense to me.

@ndelangen
Copy link
Member

My goal is to go fully CD!

But right now that's just not going to work.

  • Test coverage is too low.
  • There are no integration tests.
  • There are no flow tests.
  • Most addons are no tested at all.

I figured updating docs continuously already wouldn't be such a big deal. I was wrong, it's causing people to be confused when upgrading.

So yes, for right now I suggest to stop auto-deploying docs, and deploy manually (scripted) when performing a release.

@tmeasday
Copy link
Member

Having a deploy branch or merging PRs to a different branch will have that effect btw.

@tmeasday
Copy link
Member

I think this solves it: #1357

@ndelangen
Copy link
Member

ndelangen commented Jul 5, 2017

Yes, We talked about this on slack:

  • Docs should be deployed to separated instances per release.
  • We will introduce a dropdown for selecting a release
  • A deployed instance will do a periodic back-end call to something like https://storybook.js.org/versions.json to get the latest list of versions to display in the dropdown.

//cc @shilman

@stale
Copy link

stale bot commented Nov 14, 2017

Hi everyone! Seems like there hasn't been much going on in this issue lately. If there are still questions, comments, or bugs, please feel free to continue the discussion. Unfortunately, we don't have time to get to every issue. We are always open to contributions so please send us a pull request if you would like to help. Inactive issues will be closed after 60 days. Thanks!

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

No branches or pull requests

3 participants