Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

chore(test): move expected_conditions_spec off of the control flow #5013

Merged
merged 4 commits into from
Nov 9, 2018
Merged

chore(test): move expected_conditions_spec off of the control flow #5013

merged 4 commits into from
Nov 9, 2018

Conversation

CrispusDH
Copy link
Contributor

@cnishina , could you take a look at this PR, please

@CrispusDH
Copy link
Contributor Author

Reference #4995

@cnishina
Copy link
Member

cnishina commented Nov 8, 2018

The travis test failing is because of the ciNg2 test. I hope to have this resolved. It might require one more rebase. I'll keep you posted.

Copy link
Member

@cnishina cnishina left a comment

Choose a reason for hiding this comment

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

Please remove spaces on empty lines.

spec/basic/expected_conditions_spec.js Outdated Show resolved Hide resolved
spec/basic/expected_conditions_spec.js Outdated Show resolved Hide resolved
spec/basic/expected_conditions_spec.js Outdated Show resolved Hide resolved
@cnishina cnishina merged commit 601bd53 into angular:selenium4 Nov 9, 2018
@cnishina
Copy link
Member

cnishina commented Nov 9, 2018

Thank you for cleaning all those spaces up!

@CrispusDH
Copy link
Contributor Author

Thank you for giving me possibility to contribute :)

@CrispusDH CrispusDH deleted the expected_conditions_spec_v2 branch November 9, 2018 06:44
cnishina pushed a commit to cnishina/protractor that referenced this pull request Dec 19, 2018
cnishina pushed a commit to cnishina/protractor that referenced this pull request Mar 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants