Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(iedriver): if downloading x64, use x64 version on start command #173

Merged
merged 1 commit into from
Dec 14, 2016

Conversation

cnishina
Copy link
Member

@cnishina cnishina commented Dec 9, 2016

  • clang formatting

closes #147

@cnishina cnishina force-pushed the fix_iedriver branch 3 times, most recently from 3200136 to 965eeb4 Compare December 13, 2016 20:22
Copy link
Contributor

@heathkit heathkit left a comment

Choose a reason for hiding this comment

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

Looks like this also has some clang-format changes from the other CL.

binaries[IEDriver.id].arch = 'Win32';
}
args.push(
'-Dwebdriver.ie.driver=' +
path.resolve(outputDir, binaries[IEDriver.id].executableFilename(osType)));
}
if (options[Opt.EDGE]) {
if (options[Opt.EDGE] != null && options[Opt.EDGE].getString()) {
Copy link
Contributor

Choose a reason for hiding this comment

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

This could just be if (options[Opt.EDGE] && options[Opt.EDGE].getString())

@cnishina cnishina merged commit a3b46c7 into angular:master Dec 14, 2016
sjelin pushed a commit to sjelin/webdriver-manager that referenced this pull request Dec 16, 2016
sjelin pushed a commit to sjelin/webdriver-manager that referenced this pull request Dec 16, 2016
sjelin pushed a commit to sjelin/webdriver-manager that referenced this pull request Dec 20, 2016
sjelin pushed a commit to sjelin/webdriver-manager that referenced this pull request Dec 20, 2016
@cnishina cnishina deleted the fix_iedriver branch January 7, 2017 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

webdriver manager adds IEDriver for x64 but tries to launch x86
3 participants