Skip to content
This repository has been archived by the owner on Jun 17, 2022. It is now read-only.

0.60 and tooling #120

Merged
merged 17 commits into from
Jul 3, 2019
Merged

0.60 and tooling #120

merged 17 commits into from
Jul 3, 2019

Conversation

turnrye
Copy link
Member

@turnrye turnrye commented Apr 20, 2019

This PR adds some improvements to the changelog generator, plus the 0.60 changelog that was dogfooded using the tool.

One important note about this: something about the 0.59 -> 0.60 branch comparison on Github has broken their API (unrelated to my changes). It is not returning all of the results. This MR has been WIP for a long, long time, and I've nudged many community members to check this for any important missing changes. I anticipate however that not everyone got that message, and therefore we will have various corrections to the changelog after this ships. I'm ready to make those fixes, and moving into the future release I plan to more directly tackle the API issue that we encountered.

CHANGELOG.md Outdated Show resolved Hide resolved
@turnrye turnrye force-pushed the 0.60-and-tooling branch 2 times, most recently from 5320b16 to e2129c1 Compare June 3, 2019 02:24
@kelset
Copy link
Member

kelset commented Jun 17, 2019

hey Ryan, I was looking through the changelog and I couldn't find a reference to this commit facebook/react-native@a2285b1

Which is the one bringing "feature parity" to the inline views, that were possible in iOS from a long time (via facebook/react-native@486dbe4).

While looking into this, I also found this other commit facebook/react-native@a2a03bc which gives a few important insights:

Note that some examples are only supported on iOS because they rely on the inline view being able to size to content. Android's implementation requires that a width and height be specified on the inline view.

(but this is something that probably should live in the docs)

And also a whole list of limitations.
So probably worth writing something like "brings feature parity but be aware that it has some limitations", and maybe the link to the commit?

(more details on the whole thing here facebook/react-native#25197 (comment))

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@kelset
Copy link
Member

kelset commented Jul 1, 2019

We should probably mention something around the use_frameworks! issue -> facebook/react-native#25349

maybe we could have a line at the start saying something like "We are aware that rhe changes in 0.60.0 to integrate CocoaPods by default are causing build failures if your Podfile includes use_frameworks!." and tell them to check the issue for details of workarounds and mention that there is a PR in the works to fix it but that it will likely only land for 61

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@turnrye turnrye changed the title WIP: 0.60 and tooling 0.60 and tooling Jul 3, 2019
@kelset kelset merged commit d61ad4f into master Jul 3, 2019
@kelset kelset deleted the 0.60-and-tooling branch July 3, 2019 13:02
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.

8 participants