Skip to content
This repository was archived by the owner on Feb 26, 2024. It is now read-only.

chore: revise to conform to QS reboot in angular.io PR #2762 #287

Merged
merged 1 commit into from
Nov 23, 2016

Conversation

wardbell
Copy link
Contributor

@wardbell wardbell commented Nov 18, 2016

Coordinate with angular.io "QuickStart Reboot" PR #2782

@wardbell wardbell force-pushed the quickstart-reboot branch 2 times, most recently from 8db469b to 541cde7 Compare November 18, 2016 02:47
@filipesilva
Copy link
Contributor

@wardbell "pree2e": "npm run webdriver:update" needs to be in package.json, and can be simplified to "pree2e": "webdriver:update" I think.

Without it, we don't have a chrome driver for Protractor to use when communication with chrome. That is the source of this error message in the CI: https://travis-ci.org/angular/quickstart/builds/176909767#L780

@wardbell wardbell force-pushed the quickstart-reboot branch 3 times, most recently from 4ab6761 to d5a49ec Compare November 20, 2016 18:03
@wardbell wardbell force-pushed the quickstart-reboot branch 2 times, most recently from 34cc3f5 to 3d000f2 Compare November 22, 2016 23:50
@wardbell
Copy link
Contributor Author

wardbell commented Nov 23, 2016

Deleted "webdriver:update" from package.json and folded its work into "pree2e": "webdriver-manager update" which updates webdriver prior to e2e tests every time you say npm run e2e.

This works because npm itself detects any pre... and post... tasks and runs them appropriately and automatically

@wardbell wardbell merged commit 9f2e627 into master Nov 23, 2016
@wardbell wardbell deleted the quickstart-reboot branch November 23, 2016 00:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants