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

Add note to 'unreleased' CHANGELOG about deprecating Bower #11262

Merged

Conversation

flarnie
Copy link
Contributor

@flarnie flarnie commented Oct 17, 2017

what is the change?:
We will no longer release new versions of React to Bower, and we should
announce that as part of our CHANGELOG.

why make this change?:
We decided on this as a team.

test plan:
Visual inspection and spell check. :)

issue:
Just follow-up for #11223

**what is the change?:**
We will no longer release new versions of React to Bower, and we should
announce that as part of our CHANGELOG.

**why make this change?:**
We decided on this as a team.

**test plan:**
Visual inspection and spell check. :)

**issue:**
Just follow-up for facebook#11223
CHANGELOG.md Outdated
@@ -8,6 +8,7 @@
### All Packages

* Fix an accidental extra global variable in the UMD builds. ([@gaearon](https://github.com/gaearon) in [#10935](https://github.com/facebook/react/pull/10935))
* Deprecate use of Bower for accessing React packages. We will no longer be using Bower. ([@bvaughn](https://github.com/bvaughn) in [#11223](https://github.com/facebook/react/pull/11223))
Copy link
Collaborator

@gaearon gaearon Oct 17, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sounds like something we've done in React code itself.
It also sounds like existing Bower users will be negatively affected.

I would instead add a separate section like

Discontinuing Bower Releases

Starting with 16.1.0, we will no longer be publishing new releases on Bower. You can continue using Bower for old releases, or point your Bower configs to the React UMD builds hosted on unpkg that mirror npm releases and will continue to be updated.

I don't think attribution is needed here since it's a team decision and more of a policy change than a behavior change.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea! Will fix.

CHANGELOG.md Outdated
@@ -37,6 +37,11 @@
* Fix multiple `setState()` calls in `componentWillMount()` in shallow renderer. ([@Hypnosphi](https://github.com/Hypnosphi) in [#11167](https://github.com/facebook/react/pull/11167))
* Add back support for running in production mode. ([@gaearon](https://github.com/gaearon) in [#11112](https://github.com/facebook/react/pull/11112))


### Discontinuing Bower Releases
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's put it on top to make it hard to miss?

@flarnie
Copy link
Contributor Author

flarnie commented Oct 18, 2017

screen shot 2017-10-17 at 5 13 43 pm

Copy link
Contributor

@bvaughn bvaughn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yay! Thanks for doing this, Flarnie!

@flarnie flarnie merged commit 1019346 into facebook:master Oct 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants