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

tools: add polyfilled option to prefer-primordials rule #55318

Merged
merged 2 commits into from
Oct 17, 2024

Conversation

aduh95
Copy link
Contributor

@aduh95 aduh95 commented Oct 8, 2024

Mostly relevant for the v22.x branch now that #55276 has landed on main and won't land on v22.x. With this change, it's going to make backporting changes to v22.x easier, helping backporters pin point which commits need fixing when backporting to the v22.x staging branch without needing to run the whole test suite.

@aduh95 aduh95 added the backport-requested-v22.x PRs awaiting manual backport to the v22.x-staging branch. label Oct 8, 2024
@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. labels Oct 8, 2024
Copy link

codecov bot commented Oct 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.41%. Comparing base (9a9409f) to head (9328379).
Report is 277 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #55318   +/-   ##
=======================================
  Coverage   88.41%   88.41%           
=======================================
  Files         652      652           
  Lines      186796   186759   -37     
  Branches    36103    36106    +3     
=======================================
- Hits       165154   165126   -28     
+ Misses      14909    14904    -5     
+ Partials     6733     6729    -4     

see 41 files with indirect coverage changes

@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Oct 9, 2024
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 9, 2024
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@aduh95 aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 17, 2024
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 17, 2024
@nodejs-github-bot nodejs-github-bot merged commit 5e5af29 into nodejs:main Oct 17, 2024
63 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 5e5af29

@aduh95 aduh95 deleted the polyfilled-primordials branch October 17, 2024 10:39
aduh95 added a commit to aduh95/node that referenced this pull request Oct 17, 2024
aduh95 added a commit that referenced this pull request Oct 19, 2024
@aduh95 aduh95 mentioned this pull request Oct 24, 2024
louwers pushed a commit to louwers/node that referenced this pull request Nov 2, 2024
aduh95 added a commit to aduh95/node that referenced this pull request Nov 16, 2024
@aduh95 aduh95 added backport-open-v20.x Indicate that the PR has an open backport backport-open-v22.x Indicate that the PR has an open backport and removed backport-requested-v22.x PRs awaiting manual backport to the v22.x-staging branch. labels Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. backport-open-v20.x Indicate that the PR has an open backport backport-open-v22.x Indicate that the PR has an open backport needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants