Skip to content

Align resolve function names to have similiar namings - #471

Closed
woehrl01 wants to merge 2 commits into
react:masterfrom
woehrl01:align-resolve-names
Closed

woehrl01 wants to merge 2 commits into
react:masterfrom
woehrl01:align-resolve-names

Conversation

@woehrl01

@woehrl01 woehrl01 commented Mar 9, 2017

Copy link
Copy Markdown
Contributor

We have some resolve functions with YG**Resolve and others named YGResolve**. This changes both to be named like the later one, as I think this is the grammatically better readable one.

@emilsjolander

Copy link
Copy Markdown
Contributor

Thanks for improving the code quality!

@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

facebook-github-bot pushed a commit to react/react-native that referenced this pull request Mar 10, 2017
Summary:
We have some resolve functions with ```YG**Resolve``` and others named ```YGResolve**```. This changes both to be named like the later one, as I think this is the grammatically better readable one.
Closes react/yoga#471

Differential Revision: D4688997

Pulled By: emilsjolander

fbshipit-source-id: 38b5f84d4d39ed3effedf08188085b9efd96b4ce
vincentriemer pushed a commit to vincentriemer/yoga-dom that referenced this pull request May 9, 2018
Summary:
We have some resolve functions with ```YG**Resolve``` and others named ```YGResolve**```. This changes both to be named like the later one, as I think this is the grammatically better readable one.
Closes react/yoga#471

Differential Revision: D4688997

Pulled By: emilsjolander

fbshipit-source-id: 38b5f84d4d39ed3effedf08188085b9efd96b4ce
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.

3 participants