Skip to content

Conversation

@Rob--W
Copy link
Member

@Rob--W Rob--W commented Jul 8, 2025

Fixes #3418.

For context, see #3418 (comment)

@Rob--W Rob--W requested a review from rpl July 8, 2025 18:49
Copy link
Member

@rpl rpl left a comment

Choose a reason for hiding this comment

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

@Rob--W lgtm! ❤️ thanks for digging deep into the reasons behind this regression hit on Node 24 (and for having looked as far as to find a reasonable workaround that would work also for nodejs >= 18 while that nodejs version is still in the supported range in the web-ext manifest and part of our CI job).

@Rob--W Rob--W merged commit 8bae585 into mozilla:master Jul 10, 2025
4 checks passed
klappradla added a commit to bitcrowd/tickety-tick that referenced this pull request Oct 27, 2025
There's a bug in previous web-ext packages which make them incompatible
with current versions of NodeJS.

So we need version 8.9.0 in order to craft a release.

See:
- mozilla/web-ext#3418
- mozilla/web-ext#3466
- https://github.com/mozilla/web-ext/releases/tag/8.9.0
klappradla added a commit to bitcrowd/tickety-tick that referenced this pull request Oct 27, 2025
There's a bug in previous web-ext packages which make them incompatible
with current versions of NodeJS.

So we need version 8.9.0 in order to craft a release.

See:
- mozilla/web-ext#3418
- mozilla/web-ext#3466
- https://github.com/mozilla/web-ext/releases/tag/8.9.0
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.

Unsupported file type, please upload a supported file (.crx, .xpi, .zip).

2 participants