Skip to content

Conversation

aleen42
Copy link

@aleen42 aleen42 commented Mar 9, 2021

As we knew Microsoft has used a new Edge based on Chromium by default, and I want your plugin to try to detect via the plugin @chiragrupani/karma-chromium-edge-launcher according to karma-runner/karma-edge-launcher#11.

At the same time, it may be a better way for compatibility when integrating this plugin with karma-edge-launcher.

@aleen42 aleen42 force-pushed the chromium-edge branch 4 times, most recently from 681b785 to ae3b6da Compare March 9, 2021 08:13
@aleen42 aleen42 changed the title Support Chromium Edge with plugin @chiragrupani/karma-chromium-edge-launcher Detect Edge with the plugin @chiragrupani/karma-chromium-edge-launcher Mar 9, 2021
@aleen42 aleen42 force-pushed the chromium-edge branch 2 times, most recently from db286ef to 9d83604 Compare March 9, 2021 08:23
@aleen42 aleen42 changed the title Detect Edge with the plugin @chiragrupani/karma-chromium-edge-launcher Detect Chromium Edge with the plugin @chiragrupani/karma-chromium-edge-launcher Mar 9, 2021
@aleen42 aleen42 force-pushed the chromium-edge branch 2 times, most recently from 9101780 to 1058fb7 Compare March 9, 2021 08:44
@XhmikosR
Copy link

@litixadmin friendly ping. This would be very useful since Edge CHromium is now default installed on Windows.

@4kochi
Copy link
Collaborator

4kochi commented Jun 18, 2021

The repo is not maintained any more. Feel free to fork the repo.

@aleen42 aleen42 force-pushed the chromium-edge branch 2 times, most recently from 95e7d9a to 325d5e5 Compare January 17, 2025 08:10
@XhmikosR
Copy link

@aleen42 could you maybe publish your changes on npm?

@nickserv
Copy link
Contributor

If so, I can investigate if the fork can be used for karma-runner/karma-edge-launcher#11. If it's simple enough I'll try it, otherwise I'll be open to PRs.

@aleen42
Copy link
Author

aleen42 commented Jan 23, 2025

@XhmikosR You can try @aleen42/karma-detect-browsers@~2.3.4-beta (ref: https://www.npmjs.com/package/@aleen42/karma-detect-browsers), @nickserv it can be used which I have tested.

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.

4 participants