Skip to content

Tests: Run tests in Edge in IE mode in GitHub Actions - #5540

Merged
mgol merged 1 commit into
jquery:mainfrom
mgol:edge-ie-mode
Sep 9, 2024
Merged

mgol merged 1 commit into
jquery:mainfrom
mgol:edge-ie-mode

Conversation

@mgol

@mgol mgol commented Aug 25, 2024

Copy link
Copy Markdown
Member

Summary

While Edge in IE mode is not guaranteed to match IE 11 in every aspect, in practice it generally does. Testing in this mode in GitHub Actions will allow us to catch most IE-breaking issues at the PR level.

This change also adds missing npm scripts: test:chrome, test:edge & test:ie.

I tested it on a Windows 11 for ARM VM on macOS and in GitHub Actions on my fork - example run: https://github.com/mgol/jquery/actions/runs/10533747178/job/29190235999?pr=15

Checklist

@mgol
mgol requested a review from timmywil August 25, 2024 23:09
@mgol mgol self-assigned this Aug 25, 2024
@mgol

mgol commented Aug 25, 2024

Copy link
Copy Markdown
Member Author

Interesting that the Node / test:ie - IE (pull_request) check run on this PR... I thought actions are supposed to run in a version from the target branch? 🤔

@mgol mgol changed the title Tests: Support tests in Edge in IE mode Tests: Run tests in Edge in IE mode in GitHub Actions Aug 26, 2024
@mgol mgol added the Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. label Sep 6, 2024
@mgol
mgol requested review from timmywil and removed request for timmywil September 6, 2024 12:42
While Edge in IE mode is not guaranteed to match IE 11 in every aspect,
in practice it generally does. Testing in this mode in GitHub Actions
will allow us to catch most IE-breaking issues at the PR level.

This change also adds missing npm scripts: `test:chrome`, `test:edge`
& `test:ie`.
@timmywil timmywil removed the Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. label Sep 9, 2024
@mgol mgol removed the Needs review label Sep 9, 2024
@mgol mgol added this to the 4.0.0 milestone Sep 9, 2024
@mgol
mgol merged commit 6d78c07 into jquery:main Sep 9, 2024
@mgol
mgol deleted the edge-ie-mode branch September 9, 2024 16:24
mgol added a commit that referenced this pull request Sep 9, 2024
While Edge in IE mode is not guaranteed to match IE 11 in every aspect,
in practice it generally does. Testing in this mode in GitHub Actions
will allow us to catch most IE-breaking issues at the PR level.

This change also adds missing npm scripts: `test:chrome`, `test:edge`
& `test:ie`.

Closes gh-5540

(cherry picked from commit 6d78c07)
@mgol mgol modified the milestones: 4.0.0, 3.7.2 Sep 9, 2024
@mgol

mgol commented Sep 9, 2024

Copy link
Copy Markdown
Member Author

Landed on main in 6d78c07 and on 3.x-stable in fb281ca.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

2 participants