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

chore(typescript): driverProviders converted to typescript #3142

Merged
merged 1 commit into from
Apr 21, 2016

Conversation

cnishina
Copy link
Contributor

No description provided.

@juliemr
Copy link
Member

juliemr commented Apr 21, 2016

Travis errors are legitimate.

@@ -1 +1,11 @@
declare var browser: any;

declare class WebDriver {
Copy link
Member

Choose a reason for hiding this comment

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

I'm in favor of everything that's required from WebDriver being put in a namespace.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup, after seeing your pr... I am going to change mine to match.

@cnishina
Copy link
Contributor Author

#1203

@juliemr
Copy link
Member

juliemr commented Apr 21, 2016

Looks good! Merge it.

@cnishina cnishina merged commit a4c2fd4 into angular:master Apr 21, 2016
@cnishina cnishina deleted the pr_TSallTheThings branch April 26, 2016 22:41
@cnishina cnishina mentioned this pull request Jun 27, 2016
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.

3 participants