Skip to content

Support to MicrosoftEdge on chromium #1202

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

Merged
merged 16 commits into from
Feb 9, 2021
Merged

Support to MicrosoftEdge on chromium #1202

merged 16 commits into from
Feb 9, 2021

Conversation

v1-wizard
Copy link
Contributor

@v1-wizard v1-wizard commented Feb 2, 2021

Description

This PR adds support to MicrosoftEdge containerization.

Motivation and Context

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist

  • I have read the contributing document.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@CLAassistant
Copy link

CLAassistant commented Feb 2, 2021

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@diemol diemol left a comment

Choose a reason for hiding this comment

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

Thanks for this, @v1-wizard!
I haven't reviewed everything, but it looks good!

Seems that the commits are not associated with your GitHub user, could you please figure that out, sign the CLA, and then we can start working on merging this?

@diemol
Copy link
Member

diemol commented Feb 2, 2021

Please disregard my previous comment, I see you already fixed it.

@v1-wizard
Copy link
Contributor Author

v1-wizard commented Feb 2, 2021

@diemol hi, nice to meet you. I see that one of the test is failed, but actually make test_video work perfectly on my laptop (sounds, like tipical programmer joke, lol). Could you help me with it?

@diemol
Copy link
Member

diemol commented Feb 4, 2021

Sure, let me have a look and see what is going on. I probably need a couple of days to finish something first.

@v1-wizard
Copy link
Contributor Author

@diemol Hi, I found and fixed whole problem with tests. =)

@diemol
Copy link
Member

diemol commented Feb 9, 2021

@diemol Hi, I found and fixed whole problem with tests. =)

😆 I just saw the commit that made the tests pass, sometimes it is the little things!

Thank you for taking the time and making all tests pass! I will merge this soon.

Copy link
Member

@diemol diemol left a comment

Choose a reason for hiding this comment

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

Thank you, @v1-wizard!

@diemol diemol merged commit ab97220 into SeleniumHQ:trunk Feb 9, 2021
@diemol
Copy link
Member

diemol commented Feb 9, 2021

I need to correct some dates, and then I will release new images tomorrow.

@v1-wizard v1-wizard deleted the edge-support branch February 10, 2021 16:00
pedromctech pushed a commit to pedromctech/docker-selenium that referenced this pull request Mar 11, 2021
* Images for MicrosoftEdge with chromium was added.

* Example for StandaloneEdge was added.

* Documentation was fixed according to changes.

* * Timeout to waiting pods was upped
* Upload artifact with edge_video was added to workflows

* * Timeout to waiting pods was returned back
* The selecting of right driver version for edge was fixed

* * Rebuild without cache to update image after fix

* * Dockerfile was regenerated

* * Add missed dependencies to edge targets in Makefile

* * Up version

* * test_video was fixed
* timeout to waiting pod of selenium-chrome-node was apped in k8s-deploy-test

* Update README.md

* Update README.md

Co-authored-by: Diego Molina <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants