-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Add async / await example to page object docs #4675
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
1 similar comment
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
I signed it!
…On 24 January 2018 at 19:40, googlebot ***@***.***> wrote:
Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 *Please visit https://cla.developers.google.com/
<https://cla.developers.google.com/> to sign.*
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.
------------------------------
- If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address on your commit.
Check your existing CLA data <https://cla.developers.google.com/clas>
and verify that your email is set on your git commits
<https://help.github.com/articles/setting-your-email-in-git/>.
- If your company signed a CLA, they designated a Point of Contact who
decides which employees are authorized to participate. You may need to
contact the Point of Contact for your company and ask to be added to the
group of authorized contributors. If you don't know who your Point of
Contact is, direct the project maintainer to go/cla#troubleshoot. The email
used to register you as an authorized contributor must be the email used
for the Git commit.
- In order to pass this check, please resolve this problem and have
the pull request author add another comment and the bot will run again. If
the bot doesn't comment, it means it doesn't think anything has changed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#4675 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AASCk3CyBg65TwDThC9cyiGeqdS7cGjZks5tN2sUgaJpZM4RrpVR>
.
--
https:/linkedin.com/in/antgel
https://resume.antgel.org/
https://about.me/antonygelberg
https://antgel.org/
|
CLAs look good, thanks! |
1 similar comment
CLAs look good, thanks! |
Would you mind updating the comment according to https://github.com/angular/protractor/blob/master/CONTRIBUTING.md Thanks a lot! |
Also, Maybe we can add more description or examples to show that? |
@qiyigg I updated the comment (I had read the contributing doc, but I wrongly assumed it wasn't relevant for docs), and added another example for |
Merged. thanks! |
Changed old
getGreeting
togetGreetingText
so that I could usegetGreeting
for the non-async example of returning the object. As requested #4637.