-
Notifications
You must be signed in to change notification settings - Fork 46.9k
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
Stop generating gh-pages during release, update message #5147
Conversation
var done = this.async(); | ||
|
||
grunt.file.copy('build/react-' + VERSION + '.zip', 'docs/downloads/react-' + VERSION + '.zip'); | ||
grunt.file.copy('build/react.js', 'docs/js/react.js'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this part still important?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
… yea, it is. oops
3113b13
to
f7a3ac2
Compare
@zpao updated the pull request. |
@@ -134,8 +107,10 @@ function msg() { | |||
var steps = [ | |||
'Still todo:', | |||
'* put files on CDN', | |||
'* add starter pack (git add -f docs/downloads/react-version.zip)', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You also need to commit, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea but I figured the people doing this (me and you) are smart enough to know that. This isn't meant to be a complete step by step but a gentle reminder
Stop generating gh-pages during release, update message
We don't push the gh-pages branch directly anymore so get rid of that code from our release process.