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

Added double newline delimiters to the following functions/files: #8777

Merged
merged 1 commit into from
Jan 31, 2017

Conversation

richiethomas
Copy link
Contributor

@richiethomas richiethomas commented Jan 13, 2017

Added double newline delimiters to the following functions/files:

        * 'beginLifeCycleTimer' function of ReactDebugTool.js
        * 'bindAutoBindMethod' function of ReactClass.js
        * 'warnNoop' function of ReactServerUpdateQueue.js
        * 'getInternalInstanceReadyForUpdate' function of ReactUpdateQueue.js
        * 'warnNoop' function of ReactNoopUpdateQueue.js
        * 'getDeclarationErrorAddendum' function of ReactDOMComponent.js
        * 'getSourceInfoErrorAddendum' function of ReactElementValidator.js
        * 'getDeclarationErrorAddendum' function of instantiateReactComponent.js and ReactElementValidator.js
        * 'traverseAllChildrenImpl' function of traverseAllChildren.js
        * 'attachRef' function of ReactRef.js
        * 'mountIndeterminateComponent' function of ReactFiberBeginWork.js
        * 'createFiberFromElementType' function of ReactFiber.js
        * 'getDeclarationErrorAddendum' function of ReactDOMSelect.js
        * 'unmountComponentAtNode' function of ReactMount.js
        * 'getDeclarationErrorAddendum' function of ReactControlledValuePropTypes.js
        * 'checkRenderMessage' function of CSSPropertyOperations.js
        * 'getDeclarationErrorAddendum' function of ReactDomFiberSelect.js
        * 'getCurrentComponentErrorInfo' function in 'ReactElementValidator'
        * 'getDeclarationErrorAddendum' function in ReactDOMFiberComponent.js

Fixes #8719

            * 'beginLifeCycleTimer' function of ReactDebugTool.js
            * 'bindAutoBindMethod' function of ReactClass.js
            * 'warnNoop' function of ReactServerUpdateQueue.js
            * 'getInternalInstanceReadyForUpdate' function of ReactUpdateQueue.js
            * 'warnNoop' function of ReactNoopUpdateQueue.js
            * 'getDeclarationErrorAddendum' function of ReactDOMComponent.js
            * 'getSourceInfoErrorAddendum' function of ReactElementValidator.js
            * 'getDeclarationErrorAddendum' function of instantiateReactComponent.js and ReactElementValidator.js
            * 'traverseAllChildrenImpl' function of traverseAllChildren.js
            * 'attachRef' function of ReactRef.js
            * 'mountIndeterminateComponent' function of ReactFiberBeginWork.js
            * 'createFiberFromElementType' function of ReactFiber.js
            * 'getDeclarationErrorAddendum' function of ReactDOMSelect.js
            * 'unmountComponentAtNode' function of ReactMount.js
            * 'getDeclarationErrorAddendum' function of ReactControlledValuePropTypes.js
            * 'checkRenderMessage' function of CSSPropertyOperations.js
            * 'getDeclarationErrorAddendum' function of ReactDomFiberSelect.js
            * 'getCurrentComponentErrorInfo' function in 'ReactElementValidator'
            * 'getDeclarationErrorAddendum' function in ReactDOMFiberComponent.js
@richiethomas
Copy link
Contributor Author

@gaearon what's the procedure for requesting code reviews if no one has offered one in awhile? Should I continue to be patient or is there a way to bump this?

@gaearon
Copy link
Collaborator

gaearon commented Jan 28, 2017

Sorry I left it hanging, I’ll jump on it next week.

@aweary
Copy link
Contributor

aweary commented Jan 28, 2017

@gaearon I can give it a review if you're busy with other stuff right now

@gaearon
Copy link
Collaborator

gaearon commented Jan 28, 2017

That would be cool, thanks.

@aweary aweary assigned aweary and unassigned gaearon Jan 28, 2017
Copy link
Contributor

@aweary aweary left a comment

Choose a reason for hiding this comment

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

LGTM!

@aweary aweary added this to the 15-lopri milestone Jan 31, 2017
@aweary aweary merged commit 469e685 into facebook:master Jan 31, 2017
@gaearon
Copy link
Collaborator

gaearon commented Jan 31, 2017

@aweary Thanks for getting this in!

@richiethomas Thanks for the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants