Skip to content

Rebasing android patches over RN63 - #641

Merged
Anandraj (mganandraj) merged 2 commits into
microsoft:masterfrom
mganandraj:rn63droidpatches
Oct 28, 2020
Merged

Anandraj (mganandraj) merged 2 commits into
microsoft:masterfrom
mganandraj:rn63droidpatches

Conversation

@mganandraj

@mganandraj Anandraj (mganandraj) commented Oct 28, 2020 •

Copy link
Copy Markdown

Rebased relevant patches over RN63 codebase
Removed the dependencies on checked-in version of folly, glog & double-conversion

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

Rebased relevant patches over RN63 codebase
Removed the dependencies on checked-in version of folly, glog & double-conversion

Changelog

Rebased relevant patches over RN63 codebase
Removed the dependencies on checked-in version of folly, glog & double-conversion

[CATEGORY] [TYPE] - Message

Test Plan

Office experiences are yet to be tested.

Microsoft Reviewers: Open in CodeFlow

…encies on checked-in version of folly, glog & double-conversion
@pull-bot

Pull Request Analyser (pull-bot) commented Oct 28, 2020 •

Copy link
Copy Markdown
Messages
📖

📋 Verify Changelog Format - A changelog entry has the following format: [CATEGORY] [TYPE] - Message.

DetailsCATEGORY may be:
  • General
  • macOS
  • iOS
  • Android
  • JavaScript
  • Internal (for changes that do not need to be called out in the release notes)

TYPE may be:

  • Added, for new features.
  • Changed, for changes in existing functionality.
  • Deprecated, for soon-to-be removed features.
  • Removed, for now removed features.
  • Fixed, for any bug fixes.
  • Security, in case of vulnerabilities.

MESSAGE may answer "what and why" on a feature level. Use this to briefly tell React Native users about notable changes.

Generated by 🚫 dangerJS against 19e4cb6

@mganandraj
Anandraj (mganandraj) marked this pull request as ready for review October 28, 2020 04:40

@alloy Eloy Durán (alloy) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sweet, thanks! I’ve re-enabled the related CI test and it passes 👍

One question, can we remove the follybuild, glog, and double-conversion directories in the root of the repo too?

@alloy Eloy Durán (alloy) mentioned this pull request Oct 28, 2020
20 of 25 tasks
@mganandraj
Anandraj (mganandraj) merged commit 5c4f68b into microsoft:master Oct 28, 2020
@mganandraj
Anandraj (mganandraj) deleted the rn63droidpatches branch October 28, 2020 16:54
@alloy

Copy link
Copy Markdown
Member

Anandraj (@mganandraj) Perhaps this question slipped by:

One question, can we remove the follybuild, glog, and double-conversion directories in the root of the repo too?

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.

3 participants