Skip to content
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

fix: add alternative binary names for edge-beta #25456

Merged

Conversation

CarbonCollins
Copy link
Contributor

@CarbonCollins CarbonCollins commented Jan 13, 2023

Add alternative binary names for edge-beta and edge-canary to allow matching microsoft-edge-beta and microsoft-edge-canary Also updates snapshots to include alternate binary names

User facing changelog

Additional details

Change done due to it only being previously applied to 2 other known edge browsers in #25455

Steps to test

Have Microsoft Edge Beta and/or Microsoft Edge Canary installed on your machine with the binary names microsoft-edge-beta and microsoft-edge-canary respectivly. Cypress should now be able to discover these binaries installed on your machine without having to symlink to edge-beta or edge-canary

How has the user experience changed?

PR Tasks

  • Have tests been added/updated?
  • Has the original issue (or this PR, if no issue exists) been tagged with a release in ZenHub? (user-facing changes only)
  • Has a PR for user-facing changes been opened in cypress-documentation?
  • Have API changes been updated in the type definitions?

Add alternative binary names for edge-beta and edge-canary to allow matching microsoft-edge-beta and microsoft-edge-canary
Also updates snapshots to include alternate binary names
@CLAassistant
Copy link

CLAassistant commented Jan 13, 2023

CLA assistant check
All committers have signed the CLA.

@cypress
Copy link

cypress bot commented Jan 13, 2023

@emilyrohrbough emilyrohrbough merged commit 3eb3a16 into cypress-io:develop Jan 25, 2023
@CarbonCollins CarbonCollins deleted the add-microsoft-edge-beta branch January 26, 2023 08:09
tgriesser added a commit that referenced this pull request Jan 26, 2023
* develop: (27 commits)
  refactor: remove unused cloud routes (#25584)
  chore: fix issue template formatting issue (#25587)
  fix: fixed issue with CT + electron + run mode not exiting properly (#25585)
  chore(deps): update dependency ua-parser-js to v0.7.33 [security] (#25561)
  fix: add alternative binary names for edge-beta (#25456)
  chore: add batch execution to CloudDataSource (#22457)
  chore: End a/b campaigns for aci smart banners (#25504)
  chore: release @cypress/schematic-v2.5.0
  fix(cypress-schematic): do not disable e2e support file (#25400)
  chore: adding memory issue template (#25559)
  feat: Add Angular CT Schematic (#24374)
  chore: enforce changelog entries on PR reviews (#25459)
  chore: bump package.json to 12.4.0 [run ci] (#25556)
  feat: Add 'type' option to `.as` to store aliases by value (#25251)
  chore: release @cypress/webpack-dev-server-v3.2.3
  feat: Display line break in cy.log (#25467)
  chore: update types (#25538)
  fix: Revert "fix: adding emergency garbage collection for chromium-based browsers" (#25546)
  fix: percy - wait to take snapshot until previous tooltips are gone (#25522)
  feat: support data-qa selector in selector playground (#25475)
  ...
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Jan 27, 2023

Released in 12.4.1.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v12.4.1, please open a new issue.

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Jan 27, 2023
@AtofStryker AtofStryker removed their assignment Mar 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cypress not finding beta/dev builds of Edge on linux
5 participants