Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

Adding protractor options to fix travis failing #968

Merged
merged 1 commit into from
Oct 9, 2015

Conversation

jloveland
Copy link
Contributor

Fixing #961
travis-e2e

@lirantal
Copy link
Member

lirantal commented Oct 6, 2015

NodeJS v4 build still fails: https://travis-ci.org/meanjs/mean/jobs/83819881

@ilanbiala
Copy link
Member

@lirantal Every build is passing, am I not looking in the right place?

@codydaig
Copy link
Member

codydaig commented Oct 6, 2015

@lirantal Here is the e2e results from the Node 4.1.2 build. They seem to be passing.
https://travis-ci.org/meanjs/mean/jobs/83819881#L4386

@lirantal
Copy link
Member

lirantal commented Oct 8, 2015

Indeed they are passing. Not sure what were those errors I saw before when I inspected the v4 output from Travis.

I'm not experienced with protractor setup but the changes LGTM.

@codydaig
Copy link
Member

codydaig commented Oct 8, 2015

I'm not experienced either but LGTM

@jloveland
Copy link
Contributor Author

Can we merge this so I can make sure my e2e tests here succeed? #910

@lirantal
Copy link
Member

lirantal commented Oct 9, 2015

Yes

lirantal added a commit that referenced this pull request Oct 9, 2015
Adding protractor options to fix travis failing
@lirantal lirantal merged commit 3454cf8 into meanjs:master Oct 9, 2015
@jloveland jloveland deleted the e2e-test-error branch October 9, 2015 16:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants