-
-
Notifications
You must be signed in to change notification settings - Fork 49
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
Switch from PhantomJS to Chrome launcher #208
Conversation
If we're going to change the example I think it's worth going straight to recommending headless Chrome: |
Perhaps it's a bit premature for this to be the documented example but a link would be a start. Otherwise this is a change from headless to windowed testing. |
We'll do the switch as soon as headless Chrome is available on all platforms. PhantomJS is dead, no big arguments required 😉. |
Closing this PR for now. We are happy to accept a headless Chrome PR, once it runs stable on all platforms (Mac, Windows, Linux). |
Why not a real browser that is already there and stable? |
Well for one your example won't run in CI without |
I'd say headless Chrome is a real browser 😉. In addition to Ben's comment headless browsers have the benefit of an improved, distraction free experience for the developer. |
Setting CI is a different story, with other issues to solve. So indeed, more examples specifically targeted at CI integration would be great! For instance, I am using the Firefox runner instead in my project here: Which is also a real browser ;)
I have no experience with it, but this does not sound good at all:
So the questions to consider:
If all of these questions are positively answered, I will be happy to switch today :) If not, then using the current working runners supporting real user's browsers might be a better fit for the people coming to try this example?
In that case, perhaps, it would be better to replace it, at least temporarily? |
Ping @bendrucker @nikku Since Chrome Headless is now available I have checked on my local machine by modifying the following files, to replace phantomjs with Chrome Headless:
It will be a delight to see phantomjs removed from the package If you agree I can raise a new PR with the changes for Chrome Headless support. |
Please create the PR! |
Reasons against Phantom:
PhantomJS
downloads its huge file into thenode_module
directory every timenpm
andyarn
do not clearly report the reason for delay is thePhantomJS
installation, the user is left wondering what is going onReasons for the Chrome launcher:
Need I say more? :)