Skip to content

Conversation

mgol
Copy link
Member

@mgol mgol commented Sep 12, 2025

The previous config declared Firefox ESR testing, but the tests were never run due to clashing names with main Firefox test runs. Apart from fixing that, backport jquery/jquery#5682 so that both Firefox ESR versions can be downloaded despite using different compression algorithms.

Also, Simplify caching in CI: our setup is pretty standard, so manual configuration of actions/cache is an overkill. Relying on built-in actions/node caching will also resolve differences between caching configurations for macOS/Linux vs. Windows.

Ref gh-598
Ref jquery/jquery#5682

3.x version: #598

@mgol mgol added this to the 4.0.0 milestone Sep 12, 2025
@mgol mgol self-assigned this Sep 12, 2025
@mgol mgol force-pushed the firefox-esr-tests-fixes branch 2 times, most recently from d24eeff to b8eb451 Compare September 12, 2025 13:04
@mgol mgol marked this pull request as ready for review September 12, 2025 13:06
mgol added a commit to mgol/jquery-migrate that referenced this pull request Sep 12, 2025
The previous config declared Firefox ESR testing, but the tests were never run
due to clashing names with main Firefox test runs. Apart from fixing that,
backport jquery/jquery#5682 so that both Firefox ESR versions can be downloaded
despite using different compression algorithms.

Ref jquerygh-597
Ref jquery/jquery#5682
The previous config declared Firefox ESR testing, but the tests were never run
due to clashing names with main Firefox test runs. Apart from fixing that,
backport jquery/jquery#5682 so that both Firefox ESR versions can be downloaded
despite using different compression algorithms.

Ref jquerygh-598
Ref jquery/jquery#5682
@mgol mgol force-pushed the firefox-esr-tests-fixes branch from b8eb451 to efcf81a Compare September 12, 2025 13:08
@mgol mgol added the Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. label Sep 12, 2025
@mgol mgol requested a review from timmywil September 12, 2025 13:24
@mgol mgol force-pushed the firefox-esr-tests-fixes branch 2 times, most recently from e204bf9 to ec89ee5 Compare September 12, 2025 14:32
Our setup is pretty standard, so manual configuration of `actions/cache` is
an overkill. Relying on built-in `actions/node` caching will also resolve
differences between caching configurations for macOS/Linux vs. Windows.
@mgol mgol force-pushed the firefox-esr-tests-fixes branch from ec89ee5 to d51106c Compare September 12, 2025 14:36
@mgol mgol changed the title Build: Really test on Firefox ESR, new & old Build: Really test on Firefox ESR, new & old; simplify cache Sep 12, 2025
@mgol mgol merged commit 48021a7 into jquery:main Sep 12, 2025
21 of 22 checks passed
@mgol mgol deleted the firefox-esr-tests-fixes branch September 12, 2025 21:50
@mgol mgol removed Needs review Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. labels Sep 12, 2025
@mgol
Copy link
Member Author

mgol commented Sep 12, 2025

@timmywil Could you also approve the 3.x version of this PR? #598

mgol added a commit to mgol/jquery that referenced this pull request Sep 12, 2025
Our setup is pretty standard, so manual configuration of `actions/cache` is
an overkill. Relying on built-in `actions/node` caching will also resolve
differences between caching configurations for macOS/Linux vs. Windows.

Also, switch from `npm install` to `npm ci` in CI.

Ref jquery/jquery-migrate#597
mgol added a commit to mgol/jquery that referenced this pull request Sep 12, 2025
Our setup is pretty standard, so manual configuration of `actions/cache` is
an overkill. Relying on built-in `actions/node` caching will also resolve
differences between caching configurations for macOS/Linux vs. Windows.

Also, switch from `npm install` to `npm ci` in CI.

Ref jquery/jquery-migrate#597
mgol added a commit to mgol/jquery that referenced this pull request Sep 12, 2025
Our setup is pretty standard, so manual configuration of `actions/cache` is
an overkill. Relying on built-in `actions/node` caching will also resolve
differences between caching configurations for macOS/Linux vs. Windows.

Also, switch from `npm install` to `npm ci` in CI.

Ref jquery/jquery-migrate#597
Ref jquerygh-5702
mgol added a commit to mgol/jquery that referenced this pull request Sep 12, 2025
Our setup is pretty standard, so manual configuration of `actions/cache` is
an overkill. Relying on built-in `actions/node` caching will also resolve
differences between caching configurations for macOS/Linux vs. Windows.

Also, switch from `npm install` to `npm ci` in CI.

Ref jquery/jquery-migrate#597
Ref jquerygh-5703
mgol added a commit that referenced this pull request Sep 15, 2025
The previous config declared Firefox ESR testing, but the tests were never
run due to clashing names with main Firefox test runs. Apart from fixing
that, backport jquery/jquery#5682 so that both Firefox ESR versions can be
downloaded despite using different compression algorithms.

Also, Simplify caching in CI: our setup is pretty standard, so manual
configuration of `actions/cache` is an overkill. Relying on built-in
`actions/node` caching will also resolve differences between caching
configurations for macOS/Linux vs. Windows.

Closes gh-598
Ref gh-597
Ref jquery/jquery#5682
mgol added a commit to jquery/jquery that referenced this pull request Sep 15, 2025
Our setup is pretty standard, so manual configuration of `actions/cache`
is an overkill. Relying on built-in `actions/node` caching will also resolve
differences between caching configurations for macOS/Linux vs. Windows.

Also, switch from `npm install` to `npm ci` in CI.

Closes gh-5702
Ref jquery/jquery-migrate#597
Ref gh-5703
mgol added a commit to jquery/jquery that referenced this pull request Sep 15, 2025
Our setup is pretty standard, so manual configuration of `actions/cache`
is an overkill. Relying on built-in `actions/node` caching will also resolve
differences between caching configurations for macOS/Linux vs. Windows.

Also, switch from `npm install` to `npm ci` in CI.

Closes gh-5703
Ref gh-5702
Ref jquery/jquery-migrate#597
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.

2 participants