Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Haste] @provides -> @providesModule StaticRenderer #837

Closed
wants to merge 1 commit into from

Conversation

ide
Copy link
Contributor

@ide ide commented Apr 14, 2015

Pretty sure this shouldn't be @provides

@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 14, 2015
@ide ide force-pushed the staticrenderer branch 2 times, most recently from 070cf60 to df3ba04 Compare April 28, 2015 01:23
@ide ide force-pushed the staticrenderer branch 3 times, most recently from 7cbe98b to 927cc4a Compare May 13, 2015 23:03
@vjeux
Copy link
Contributor

vjeux commented May 14, 2015

@facebook-github-bot import

Pretty sure this shouldn't be `@provides`, since StaticRenderer is required as a module instead of with `@requires`. =)

Test Plan: Ran UIExplorer and flow.
vjeux pushed a commit to vjeux/react-native that referenced this pull request May 14, 2015
Summary:
Pretty sure this shouldn't be @provides
Closes facebook#837
Github Author: James Ide <ide@jameside.com>

Test Plan: Imported from GitHub, without a `Test Plan:` line.
@facebook-github-bot
Copy link
Contributor

Thanks for importing. If you are an FB employee go to https://our.intern.facebook.com/intern/opensource/github/pull_request/1576511432619372/int_phab to review.

@vjeux
Copy link
Contributor

vjeux commented May 14, 2015

@ide: we found the bug why it didn't update. Should be working fine for old pull requests in a few days (need to make a real diff + land it)

@ide ide closed this in b1c93bb May 14, 2015
@ide ide deleted the staticrenderer branch May 15, 2015 04:51
ayushjainrksh referenced this pull request in MLH-Fellowship/react-native Jul 2, 2020
* Update state.md

ES6 and new react version does not need super or constructor

* Update state.md

Added setInterval to componentDidMount, accordance to latest syntax
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants