Skip to content

CI: check for and warn about possible missing polyfills #15268

Merged
patmmccann merged 16 commits into
masterfrom
polyfills
Jul 8, 2026
Merged

CI: check for and warn about possible missing polyfills #15268
patmmccann merged 16 commits into
masterfrom
polyfills

Conversation

@dgirardi

@dgirardi dgirardi commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Type of change

  • CI related changes

Description of change

  • Add a build option to generate polyfills.json, a summary of corejs polyfills that would be injected if we were using babel-plugin-polyfill-corejs3;
  • add a workflow that compares the summary of head and base branches and warns about any new entries.

Example warning: dgirardi#74 (comment)

The new workflow is expected to fail until this is merged (and included in the base of new PRs).

Other information

Closes #15130

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 662999f2aa

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread .github/workflows/PR-assignment.yml Outdated
@barecheck

barecheck Bot commented Jun 30, 2026

Copy link
Copy Markdown

Barecheck - Code coverage report

Total: 96.56%

Your code coverage diff: 0.00% ▴

✅ All code changes are covered

@dgirardi dgirardi marked this pull request as draft July 1, 2026 06:42
@dgirardi dgirardi marked this pull request as ready for review July 1, 2026 13:53
@patmmccann patmmccann merged commit cc974ce into master Jul 8, 2026
126 of 127 checks passed
@patmmccann patmmccann deleted the polyfills branch July 8, 2026 10:19
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.

follows ups to build target change

2 participants