Skip to content

Convert SafeAreaView to ES Module#31344

Closed
franzon wants to merge 1 commit into
facebook:masterfrom
franzon:convert-safeareaview-to-esm
Closed

Convert SafeAreaView to ES Module#31344
franzon wants to merge 1 commit into
facebook:masterfrom
franzon:convert-safeareaview-to-esm

Conversation

@franzon
Copy link
Copy Markdown
Contributor

@franzon franzon commented Apr 12, 2021

Summary

Convert SafeAreaView to ES Module (#31330).

Changelog

[General] [Changed] - Converted SafeAreaView to ES Module.

Test Plan

Tested that npm test passes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 12, 2021
@analysis-bot
Copy link
Copy Markdown

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: a15a46c

@lunaleaps lunaleaps self-assigned this Apr 14, 2021
Copy link
Copy Markdown
Contributor

@lunaleaps lunaleaps left a comment

Choose a reason for hiding this comment

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

Other than that, this looks good to me! Thank you for working on this!!

}

module.exports = exported;
export default exported;
Copy link
Copy Markdown
Contributor

@lunaleaps lunaleaps Apr 14, 2021

Choose a reason for hiding this comment

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

nit: maybe it's clearer to rename this to SafeAreaView?

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@lunaleaps has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

1 similar comment
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@lunaleaps has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@lunaleaps merged this pull request in d9a86aa.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Apr 14, 2021
@lunaleaps
Copy link
Copy Markdown
Contributor

Thanks @franzon for your contribution!

@lunaleaps lunaleaps reopened this Apr 14, 2021
@lunaleaps lunaleaps closed this Apr 14, 2021
@lunaleaps
Copy link
Copy Markdown
Contributor

Sorry accidentally clicked "Reopen and comment" lol

@kacieb kacieb linked an issue Apr 23, 2021 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Convert SafeAreaView to ES Module

4 participants