-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
[WIP] - Support Microsoft Edge (Chromium Based) #4203
Conversation
@jennifer-shehane thanks for reviewing, I am tracking and testing the edge packages at this github repo https://github.com/YOU54F/cypress-edge and have it working in CI at the moment :) |
Need additional changes to make it work on windows |
🙄 yep, hence why I’ve not marked the windows checkboxes in the first post. |
now supporting windows edge/dev ms edge builds
|
Got Edge Dev & Edge Canary running on Windows now but I am having trouble building the binary. It gets most of the way but fails when trying to perform a symlink for the electron package. I am running git bash for windows (https://gitforwindows.org/)
How are you building it for windows? Via Appveyor? BTW - Your contributing and deployment docs are quality! and the build process is 👌 kudos to all in the team. |
@YOU54F Hmm, I don't know if I've seen that, but re-building Cypress on Windows from scratch always fixes whatever weird filesystem issue I'm having. Have you tried re-cloning Cypress and doing |
Cheers for the reply bud, I'll try it on my desktop machine when I'm home, I've got a few diff shells on that, and WSL (windows subsystem linux) so may have more luck. Will keep you posted 👍 |
@YOU54F I have confirmed it is working on Mac, could you accept https://github.com/YOU54F/cypress/pull/5 that fixes the linting errors? For some reason my CLI pull request never upload to this PR, instead they create a new branch, ughh |
ughh, Edge is not available on our https://www.appveyor.com/docs/windows-images-software/#web-browsers |
fix lint errors
@bahmutov accepted the pr now for linter errors. Chromium edge is still only available on insiders at the moment, so you would need to download and install the executable directly on a windows vm. I have been meaning to set it up with circleci now that they have windows executors |
That PR does indeed look like it would be ideal! I think there was someone on gitter trying to get chrome to recognise a browser that chrome based but didn't mention which one, so it looks like it would get many birds with one stone. Will see what I can do with regards to the windows edge on circleci with cypress 👍 got the laptop out as I have been doing some tweaks on a work project to see if we can speed up build times but that can really wait until the 9-5! |
yeah, not rush there. I think this PR is great because it has the detection logic. |
Ahhh reading my last comments, I forgot that I couldn't actually get it to build the binaries on my windows machines! |
@bahmutov - tried to setup a run in circleci, but I've ran out of credits so no dice, but got the binaries built in windows (via bootcamp on my mac) and have it working now. I have released the binaries here :- https://github.com/YOU54F/cypress/releases/download/v3.5.0/cypress_win.zip and an associated cli release here :- https://www.npmjs.com/package/@you54f/cypress instructions to use, for testing Supported versions
Instructions for Windows
|
Hey @YOU54F, any updates on this PR? |
Hello @jennifer-shehane, Edge is due out on general release near the end of the January, it was announced near the start or December. There is no news on a Linux build yet, so it will be macOS and windows only. We could release the Mac changes now and the windows changes when we have the official executable names when it goes on general release or support edge dev, beta and canary until the released versions are out? |
@YOU54F Ok, I just wanted to hear what you felt was left. There's a lot more left on our side:
These are just off the top of my head, there's a lot more that needs updating, you can see some in the Firefox PR that we would need to do for any added Browser https://github.com/cypress-io/cypress/pull/1359/files I suspect we'll have someone from our team start a new PR for this work is what I'm getting at. cc @bahmutov |
The MS Edge went into wide availability https://blogs.windows.com/windowsexperience/2020/01/15/new-year-new-browser-the-new-microsoft-edge-is-out-of-preview-and-now-available-for-download/ |
Cheers @bahmutov & @jennifer-shehane glad to help! |
Proposal for Microsoft Edge (Chromium Based) Support in Cypress
Offical Microsoft Edge Insiders page here
Currently tested
Terminal output after change (in
/packages/launcher
)