Skip to content

Conversation

HeyImChris
Copy link

@HeyImChris HeyImChris commented Dec 19, 2019

  • [X ] I am making a fix / change for the macOS implementation of react-native
  • [X ] I am making a change required for Microsoft usage of react-native

Moving the FB version merge documentation here so it's public for anyone to leverage. Adding this file is a small deviation from Fb's repo, but seems worth it as it's fittingly a markdown on how to help us not deviate from Fb's repo.

Focus areas to test

Documentation change. Tested the links and markdown at https://stackedit.io/app#

Microsoft Reviewers: Open in CodeFlow

…where it belongs and can be accessed by anyone
KeepingRecent.md Outdated
* iOS- chrishog/tomun
* Mac- chrishog/tomun
* Make sure you're pulling in a **stable** [release candidate](https://facebook.github.io/react-native/versions).
* Note that this *only* gets the newest RN version into [our fork](http://github.com/microsoft/react-native) which will then have to get pushed into downstream repositories such as [SDX-Platform](https://office.visualstudio.com/ISS/_git/sdx-platform) for its consumption. Because we have quite a few code changes in our fork, but not Facebook's master, you should expect some more changes to both downstream clients and [our fork](http://github.com/microsoft/react-native) to keep everything in working order. No newline at end of file

Choose a reason for hiding this comment

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

can probably leave this out too. This is a public repo and we don't need to mention internals that a contributor or consumer of this repro doesn't need to know

KeepingRecent.md Outdated
Comment on lines 35 to 39
* Platform owners
* Win32- acoates
* Android- acoates
* iOS- chrishog/tomun
* Mac- chrishog/tomun

Choose a reason for hiding this comment

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

per above, can leave these out or at least use github names rather than aliases

Copy link
Author

Choose a reason for hiding this comment

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

Good point about it being public. I like having some references in just in case some random person finds this and wants to know more about our platform support. Definitely makes sense to switch to our github aliases.

@HeyImChris HeyImChris merged commit e1ac1bc into master Jan 2, 2020
@acoates-ms acoates-ms deleted the chrishog/mergeDocs branch April 22, 2020 17:48
rozele pushed a commit to rozele/react-native-macos that referenced this pull request Sep 25, 2025
Summary:
Pull Request resolved: facebook#53941

Changelog: [Internal] - Update `react-native/debugger-frontend` from 8b84a77...8cce390

Resyncs `react-native/debugger-frontend` from GitHub - see `rn-chrome-devtools-frontend` [changelog](facebook/react-native-devtools-frontend@8b84a77...8cce390).

### Changelog

| Commit | Author | Date/Time | Subject |
| ------ | ------ | --------- | ------- |
| [8cce39003](facebook/react-native-devtools-frontend@8cce39003) | Alex Hunt (hello@alexhunt.dev) | 2025-09-25T15:05:50+01:00 | [Update Welcome panel assets and styling (microsoft#208)](facebook/react-native-devtools-frontend@8cce39003) |
| [7724ccb99](facebook/react-native-devtools-frontend@7724ccb99) | Alex Hunt (hello@alexhunt.dev) | 2025-09-25T08:46:01+01:00 | [Disable Perf Insights subpanel, collapse sidebar by default (microsoft#209)](facebook/react-native-devtools-frontend@7724ccb99) |
| [538c309a3](facebook/react-native-devtools-frontend@538c309a3) | Vitali Zaidman (vzaidman@gmail.com) | 2025-09-18T12:20:22+01:00 | [track when all startup resources are loaded (microsoft#205)](facebook/react-native-devtools-frontend@538c309a3) |

Reviewed By: hoxyq

Differential Revision: D83248877

fbshipit-source-id: e5d7fb009a89bd97c8aac5f0a09b5733b8249651
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants