protractor icon indicating copy to clipboard operation
protractor copied to clipboard

feat(config): support async config

Open Amy-Lynn opened this issue 6 years ago • 17 comments

Allows protractor config file export to be a promise that resolves to a config object.

Closes #5148 Closes #4749


Unfortunately, some of these tests aren't passing on my machine (on master), so I need to use the PR to run them.

Amy-Lynn avatar Feb 12 '19 03:02 Amy-Lynn

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

googlebot avatar Feb 12 '19 03:02 googlebot

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

googlebot avatar Feb 12 '19 03:02 googlebot

Noting that when I reverted my change (so should be matching master), CircleCI tests are still failing with SessionNotCreatedError: session not created: Chrome version must be between 71 and 75

Amy-Lynn avatar Feb 12 '19 05:02 Amy-Lynn

I signed it!

Amy-Lynn avatar Feb 12 '19 20:02 Amy-Lynn

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. 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.

googlebot avatar Feb 12 '19 20:02 googlebot

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. 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.

googlebot avatar Feb 12 '19 20:02 googlebot

I signed it!

Amy-Lynn avatar Feb 13 '19 01:02 Amy-Lynn

CLAs look good, thanks!

googlebot avatar Feb 13 '19 01:02 googlebot

CLAs look good, thanks!

googlebot avatar Feb 13 '19 01:02 googlebot

Please rebase. Protractor 6 has been updated and there were a lot of changes. Also please use native promises. Thanks.

cnishina avatar Mar 24 '19 16:03 cnishina

@cnishina Updated as requested

Amy-Lynn avatar Mar 25 '19 16:03 Amy-Lynn

When this will be merged? Waiting for this feature!

tymfear avatar Apr 05 '19 10:04 tymfear

Resolved a conflict I created from a previous merge. Once this passes, I'm adding it in.

cnishina avatar Apr 07 '19 09:04 cnishina

Also thanks for pinging this again @tymfear. The notification was buried. Also thanks @Amy-Lynn for this PR and the test additions.

cnishina avatar Apr 07 '19 09:04 cnishina

So once again pinging. Can we update the branch and merge it?

tymfear avatar May 06 '19 16:05 tymfear

Yes, this will. It will be released for Protractor 6. It will be available in beta soon.

cnishina avatar May 06 '19 16:05 cnishina

Yes, this will. It will be released for Protractor 6. It will be available in beta soon.

@cnishina Cool, thank you!

tymfear avatar May 06 '19 16:05 tymfear