Skip to content

Conversation

@danez
Copy link
Collaborator

@danez danez commented Jun 3, 2016

@chentsulin I also included stuff from your pull request. I hope I copied everything correctly.

@chentsulin
Copy link
Member

In my opinion, rackt-cli's release script is too complex, npm version and prepublish hook is enough. And I will not keep umd build in the source control.

@danez
Copy link
Collaborator Author

danez commented Jun 3, 2016

I think exactly the same, because this library is also not that big, but as the release is still done by @mzabriskie I don't want to break the release flow too much, unless he is fine on simplifying it.

The umd build is for bower, I have too check how publish for bower works. Imho I would drop bower, not sure if it is really used, as react-tabs is the only component that I know that has support for bower.

@danez danez merged commit bb6ac5b into reactjs:master Jun 6, 2016
@danez danez deleted the fix-tests-and-checks branch June 6, 2016 09:36
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 28, 2022
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