Skip to content

Conversation

christophpurrer
Copy link

@christophpurrer christophpurrer commented Nov 13, 2022

Please select one of the following

  • I am removing an existing difference between facebook/react-native and microsoft/react-native-macos 👍
  • I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos 👍
  • I am making a fix / change for the macOS implementation of react-native
  • I am making a change required for Microsoft usage of react-native

Summary

These files have some 'unnecessary' changes which make it hard(er) to merge in recent changes as: facebook@37e5fa3a6ce

Hence un-forking them were possible

Changelog

[macOS][Fixed] Unfork RCTLogBox

Test Plan

CircleCi for macOS and iOS

Show some beautiful logboxes ...

Screen.Recording.2022-11-13.at.10.57.19.PM.mov

@christophpurrer christophpurrer marked this pull request as ready for review November 14, 2022 06:02
@christophpurrer christophpurrer requested a review from a team as a code owner November 14, 2022 06:02
@Saadnajmi
Copy link
Collaborator

@amgleitman any opinions? I remember you ran into this same diff during one of your last merges.

These files have some 'unnecessary' changes which make it hard(er) to merge in recent changes as:
facebook@37e5fa3a6ce
Hence unforking them were possible
@Saadnajmi Saadnajmi merged commit bc87c85 into microsoft:main Nov 16, 2022
@christophpurrer christophpurrer deleted the logBoxView branch November 16, 2022 00:36
shwanton pushed a commit to shwanton/react-native-macos that referenced this pull request Feb 13, 2023
These files have some 'unnecessary' changes which make it hard(er) to merge in recent changes as:
facebook@37e5fa3a6ce
Hence unforking them were possible
shwanton pushed a commit to shwanton/react-native-macos that referenced this pull request Mar 10, 2023
These files have some 'unnecessary' changes which make it hard(er) to merge in recent changes as:
facebook@37e5fa3a6ce
Hence unforking them were possible
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants