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

Rhe dev #9368

Closed
wants to merge 749 commits into from
Closed

Rhe dev #9368

wants to merge 749 commits into from

Conversation

f2009
Copy link

@f2009 f2009 commented Jan 25, 2019

The First PR!!!

TimSpeelman and others added 30 commits January 25, 2019 17:44
…omments (vuejs#8529)

Improvements that apply to all comments:
- Consistent use of periods
- Consistent use of commas
- Consistent capitalization
- Correct use of articles (a, an, the)

Improvements to specific comments:
- Correct pluralization in the description for helper functions
- Reword for clarity and break into shorter sentences the description
for `polyfillBind` function
- Reword for clarity the description for `genStaticKeys` function
- Add a previously missing description for `looseIndexOf` function

Fixes: vuejs#8528
yyx990803 and others added 28 commits January 25, 2019 17:51
also optimize fix performance by avoiding calls to performance.now()
@posva
Copy link
Member

posva commented Jan 25, 2019

Please, use your own repos for tests, thank you :)

@posva posva closed this Jan 25, 2019
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.

None yet