-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
--select-chrome and --skip-autolaunch flags removed #2659
--select-chrome and --skip-autolaunch flags removed #2659
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like we haven't done resolveChromePath
for Mac so far. Can you add that? I guess you'll have to add those paths to the darwin priorities array as well.
Aside from that everything else looks good!
Thanks. :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! thanks ;)
]; | ||
// clang-format on | ||
|
||
return sort(installations, priorities); | ||
} | ||
|
||
function resolveChromePath() { | ||
if (canAccess(process.env.CHROME_PATH)) { | ||
return process.env.CHROME_PATH as string; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is the as string
necessary here if canAccess()
doesn't complain in the line above?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed in a6bbcab
GoogleChrome/lighthouse#2659) * --skip-autolaunch flag removed, --select-chrome flag removed * Unused chrome-launcher/ask.ts removed * Added support for CHROME_PATH env variable to replace LIGHTHOUSE_CHROMIUM_PATH * CHROME_PATH env variable documented in CLI help * Added support for CHROME_PATH on macOS
Fixes #2617