Skip to content

test: fix permission fixtures lint - #55819

Merged
aduh95 merged 1 commit into
nodejs:mainfrom
RafaelGSS:fix-permission-fixtures-lint
Nov 12, 2024
Merged

test: fix permission fixtures lint#55819
aduh95 merged 1 commit into
nodejs:mainfrom
RafaelGSS:fix-permission-fixtures-lint

Conversation

@RafaelGSS

Copy link
Copy Markdown
Member

@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Nov 11, 2024
@aduh95aduh95 added fast-track PRs that do not need to wait for 48 hours to land. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. and removed needs-ci PRs that need a full CI run. labels Nov 11, 2024
@github-actions

Copy link
Copy Markdown
Contributor

Fast-track has been requested by @aduh95. Please 👍 to approve.

@codecov

codecovBot commented Nov 11, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.41%. Comparing base (3a0968d) to head (e936180).
Report is 93 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #55819 +/- ##
==========================================
- Coverage 88.41% 88.41% -0.01% 
==========================================
Files 654 654 Lines 187811 187811 Branches 36134 36129 -5 ==========================================
- Hits 166052 166044 -8 + Misses 15008 15001 -7 - Partials 6751 6766 +15 

see 35 files with indirect coverage changes

@RafaelGSSRafaelGSS added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 12, 2024
@nodejs-github-botnodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Nov 12, 2024
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/55819
✔ Done loading data for nodejs/node/pull/55819
----------------------------------- PR info ------------------------------------
Title test: fix permission fixtures lint (#55819)
Author Rafael Gonzaga <rafael.nunu@hotmail.com> (@RafaelGSS)
Branch RafaelGSS:fix-permission-fixtures-lint -> nodejs:main
Labels test, fast-track, author ready
Commits 1
- test: fix permission fixtures lint
Committers 1
- RafaelGSS <rafael.nunu@hotmail.com>
PR-URL: https://github.com/nodejs/node/pull/55819
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/55819
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
--------------------------------------------------------------------------------
ℹ This PR was created on Mon, 11 Nov 2024 17:39:04 GMT
✔ Approvals: 1
✔ - Antoine du Hamel (@aduh95) (TSC): https://github.com/nodejs/node/pull/55819#pullrequestreview-2427864741
ℹ This PR is being fast-tracked
✘ This PR needs to wait 149 more hours to land (or 29 hours if there is one more approval) (or 0 hours if there is 1 more approval (👍) of the fast-track request from collaborators).
✔ Last GitHub CI successful
✘ No Jenkins CI runs detected
--------------------------------------------------------------------------------
✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/11796935552

@lpincalpinca added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. commit-queue Add this label to land a pull request using GitHub Actions. labels Nov 12, 2024
@aduh95
aduh95 merged commit bd0c4f8 into nodejs:mainNov 12, 2024
@aduh95

Copy link
Copy Markdown
Contributor

Landed in bd0c4f8

RafaelGSS added a commit to nodejs/changelog-maker that referenced this pull request Nov 12, 2024
RafaelGSS added a commit to nodejs/changelog-maker that referenced this pull request Nov 12, 2024
aduh95 pushed a commit that referenced this pull request Nov 16, 2024
PR-URL: #55819
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
RafaelGSS added a commit to nodejs/changelog-maker that referenced this pull request Nov 22, 2024
@ruyadorno

Copy link
Copy Markdown
Member

This commit does not land cleanly on v22.x-staging and will need manual backport in case we want it in v22.x.

@ruyadornoruyadorno added the backport-requested-v22.x PRs awaiting manual backport to the v22.x-staging branch. label Nov 27, 2024
@aduh95aduh95 removed the backport-requested-v22.x PRs awaiting manual backport to the v22.x-staging branch. label May 19, 2025
aduh95 pushed a commit that referenced this pull request May 19, 2025
PR-URL: #55819
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@ghostghost mentioned this pull request Jun 8, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.fast-trackPRs that do not need to wait for 48 hours to land.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@RafaelGSS@nodejs-github-bot@aduh95@ruyadorno@lpinca