-
Notifications
You must be signed in to change notification settings - Fork 49.5k
Add changelog entry for 19.1.1 #34021
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
Conversation
Comparing: cc01584...91a31ee Critical size changesIncludes critical production bundles, as well as any change greater than 2%:
Significant size changesIncludes any change greater than 0.2%: (No significant changes) |
The last two items seem a bit misleading to classify as "React" since they probably only affect the RN renderer (not the React package) and don't get released until someone syncs RN? Should they even be in the React changelog at all? |
I'll PR with rationale |
Yes thats fair. I only included because they were intentional bug fixes to unblock the next RN version and avoid us having to customize the renderer flags outside of the OSS releases. I'm fine with it moving from this changelog to the next entry for RN. |
See #34021 (comment). The purpose of the changelog is to communicate to React users what changed in the release. Therefore, it is important that the changelog is written oriented towards React end users. Historically this means that we omit internal-only changes, i.e. changes that have no effect on the end user behavior. If internal changes are mentioned in the changelog (e.g. if they affect end user behavior), they should be phrased in a way that is understandable to the end user — in particular, they should not refer to internal API names or concepts. We also try to group changes according to the publicly known packages. In this PR: - Make #33680 an actual link (otherwise it isn't linkified in CHANGELOG.md on GitHub). - Remove two changelog entries listed under "React" that don't affect anyone who upgrades the "React" package, that are phrased using terminology and internal function names unfamiliar to React users, and that seem to be RN-specific changes (so should probably go into the RN changelog that goes out with the next renderer sync that includes these changes).
Add changelog details matching release notes: https://github.com/facebook/react/releases/tag/v19.1.1