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

doc: deprecation code for isWebAssemblyCompiledModule #50486

Conversation

marco-ippolito
Copy link
Member

@marco-ippolito marco-ippolito commented Oct 31, 2023

It was missing deprecation code

@nodejs-github-bot nodejs-github-bot added deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations. labels Oct 31, 2023
@jasnell jasnell added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 11, 2023
@nodejs-github-bot nodejs-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 11, 2023
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/50486
✔  Done loading data for nodejs/node/pull/50486
----------------------------------- PR info ------------------------------------
Title      doc: deprecation code for isWebAssemblyCompiledModule (#50486)
Author     Marco Ippolito  (@marco-ippolito)
Branch     marco-ippolito:feat/deprecate-isWebAssemblyCompiledModule -> nodejs:main
Labels     doc, deprecations
Commits    4
 - doc: deprecation code for isWebAssemblyCompiledModule
 - addded pr url
 - Update deprecations.md
 - fix
Committers 2
 - Marco Ippolito 
 - GitHub 
PR-URL: https://github.com/nodejs/node/pull/50486
Reviewed-By: Paolo Insogna 
Reviewed-By: James M Snell 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/50486
Reviewed-By: Paolo Insogna 
Reviewed-By: James M Snell 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Tue, 31 Oct 2023 10:03:58 GMT
   ✔  Approvals: 2
   ✔  - Paolo Insogna (@ShogunPanda): https://github.com/nodejs/node/pull/50486#pullrequestreview-1708249819
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/50486#pullrequestreview-1726083822
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  No git cherry-pick in progress
   ✔  No git am in progress
   ✔  No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
 * branch                  main       -> FETCH_HEAD
✔  origin/main is now up-to-date
- Downloading patch for 50486
From https://github.com/nodejs/node
 * branch                  refs/pull/50486/merge -> FETCH_HEAD
✔  Fetched commits as 83e6350b826a..3f0db4ecc604
--------------------------------------------------------------------------------
[main 25fe4458f4] doc: deprecation code for isWebAssemblyCompiledModule
 Author: Marco Ippolito 
 Date: Tue Oct 31 12:00:56 2023 +0200
 1 file changed, 18 insertions(+)
[main 68ab441ba5] addded pr url
 Author: Marco Ippolito 
 Date: Tue Oct 31 12:04:25 2023 +0200
 1 file changed, 1 insertion(+), 1 deletion(-)
[main 416bf34715] Update deprecations.md
 Author: Marco Ippolito 
 Date: Tue Oct 31 12:17:41 2023 +0200
 1 file changed, 2 insertions(+), 2 deletions(-)
[main 1e3b421ffb] fix
 Author: Marco Ippolito 
 Date: Tue Oct 31 14:16:18 2023 +0200
 1 file changed, 2 insertions(+), 2 deletions(-)
   ✔  Patches applied
There are 4 commits in the PR. Attempting autorebase.
Rebasing (2/8)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
doc: deprecation code for isWebAssemblyCompiledModule

PR-URL: #50486
Reviewed-By: Paolo Insogna paolo@cowtech.it
Reviewed-By: James M Snell jasnell@gmail.com

[detached HEAD 5cb6020934] doc: deprecation code for isWebAssemblyCompiledModule
Author: Marco Ippolito marcoippolito54@gmail.com
Date: Tue Oct 31 12:00:56 2023 +0200
1 file changed, 18 insertions(+)
Rebasing (3/8)
Rebasing (4/8)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
addded pr url

PR-URL: #50486
Reviewed-By: Paolo Insogna paolo@cowtech.it
Reviewed-By: James M Snell jasnell@gmail.com

[detached HEAD f0f9b0f266] addded pr url
Author: Marco Ippolito marcoippolito54@gmail.com
Date: Tue Oct 31 12:04:25 2023 +0200
1 file changed, 1 insertion(+), 1 deletion(-)
Rebasing (5/8)
Rebasing (6/8)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
Update deprecations.md

PR-URL: #50486
Reviewed-By: Paolo Insogna paolo@cowtech.it
Reviewed-By: James M Snell jasnell@gmail.com

[detached HEAD 0543c368a6] Update deprecations.md
Author: Marco Ippolito marcoippolito54@gmail.com
Date: Tue Oct 31 12:17:41 2023 +0200
1 file changed, 2 insertions(+), 2 deletions(-)
Rebasing (7/8)
Rebasing (8/8)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
fix

PR-URL: #50486
Reviewed-By: Paolo Insogna paolo@cowtech.it
Reviewed-By: James M Snell jasnell@gmail.com

[detached HEAD b68af47e1d] fix
Author: Marco Ippolito marcoippolito54@gmail.com
Date: Tue Oct 31 14:16:18 2023 +0200
1 file changed, 2 insertions(+), 2 deletions(-)

Successfully rebased and updated refs/heads/main.

ℹ Add commit-queue-squash label to land the PR as one commit, or commit-queue-rebase to land as separate commits.

https://github.com/nodejs/node/actions/runs/6835327454

@marco-ippolito marco-ippolito force-pushed the feat/deprecate-isWebAssemblyCompiledModule branch 2 times, most recently from 65723c3 to b97c02b Compare November 11, 2023 16:36
@marco-ippolito marco-ippolito 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. labels Nov 11, 2023
@nodejs-github-bot nodejs-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 11, 2023
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/50486
✔  Done loading data for nodejs/node/pull/50486
----------------------------------- PR info ------------------------------------
Title      doc: deprecation code for isWebAssemblyCompiledModule (#50486)
Author     Marco Ippolito  (@marco-ippolito)
Branch     marco-ippolito:feat/deprecate-isWebAssemblyCompiledModule -> nodejs:main
Labels     doc, deprecations
Commits    1
 - doc: deprecation code for isWebAssemblyCompiledModule
Committers 1
 - marco-ippolito 
PR-URL: https://github.com/nodejs/node/pull/50486
Reviewed-By: Paolo Insogna 
Reviewed-By: James M Snell 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/50486
Reviewed-By: Paolo Insogna 
Reviewed-By: James M Snell 
--------------------------------------------------------------------------------
   ⚠  Commits were pushed since the last approving review:
   ⚠  - doc: deprecation code for isWebAssemblyCompiledModule
   ℹ  This PR was created on Tue, 31 Oct 2023 10:03:58 GMT
   ✔  Approvals: 2
   ✔  - Paolo Insogna (@ShogunPanda): https://github.com/nodejs/node/pull/50486#pullrequestreview-1708249819
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/50486#pullrequestreview-1726083822
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/6836131662

@targos
Copy link
Member

targos commented Nov 12, 2023

Please correct the commit message (it's missing a verb)

@marco-ippolito marco-ippolito force-pushed the feat/deprecate-isWebAssemblyCompiledModule branch from b97c02b to 38f52f8 Compare November 13, 2023 08:26
@marco-ippolito marco-ippolito removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Nov 13, 2023
@marco-ippolito marco-ippolito added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 22, 2023
@nodejs-github-bot nodejs-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 22, 2023
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/50486
✔  Done loading data for nodejs/node/pull/50486
----------------------------------- PR info ------------------------------------
Title      doc: deprecation code for isWebAssemblyCompiledModule (#50486)
Author     Marco Ippolito  (@marco-ippolito)
Branch     marco-ippolito:feat/deprecate-isWebAssemblyCompiledModule -> nodejs:main
Labels     doc, deprecations
Commits    1
 - doc: create deprecation code for isWebAssemblyCompiledModule
Committers 1
 - marco-ippolito 
PR-URL: https://github.com/nodejs/node/pull/50486
Reviewed-By: Paolo Insogna 
Reviewed-By: James M Snell 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/50486
Reviewed-By: Paolo Insogna 
Reviewed-By: James M Snell 
--------------------------------------------------------------------------------
   ⚠  Commits were pushed since the last approving review:
   ⚠  - doc: create deprecation code for isWebAssemblyCompiledModule
   ℹ  This PR was created on Tue, 31 Oct 2023 10:03:58 GMT
   ✔  Approvals: 2
   ✔  - Paolo Insogna (@ShogunPanda): https://github.com/nodejs/node/pull/50486#pullrequestreview-1708249819
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/50486#pullrequestreview-1726083822
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/6960144937

@marco-ippolito marco-ippolito 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. labels Nov 22, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 22, 2023
@nodejs-github-bot nodejs-github-bot merged commit 3782006 into nodejs:main Nov 22, 2023
22 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 3782006

targos pushed a commit that referenced this pull request Nov 23, 2023
PR-URL: #50486
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
martenrichter pushed a commit to martenrichter/node that referenced this pull request Nov 26, 2023
PR-URL: nodejs#50486
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
lucshi pushed a commit to lucshi/node that referenced this pull request Nov 27, 2023
PR-URL: nodejs#50486
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
@RafaelGSS RafaelGSS mentioned this pull request Nov 28, 2023
RafaelGSS pushed a commit that referenced this pull request Nov 29, 2023
PR-URL: #50486
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
RafaelGSS pushed a commit that referenced this pull request Nov 30, 2023
PR-URL: #50486
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
UlisesGascon pushed a commit that referenced this pull request Dec 11, 2023
PR-URL: #50486
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
@UlisesGascon UlisesGascon mentioned this pull request Dec 12, 2023
UlisesGascon pushed a commit that referenced this pull request Dec 13, 2023
PR-URL: #50486
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
UlisesGascon pushed a commit that referenced this pull request Dec 15, 2023
PR-URL: #50486
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
UlisesGascon pushed a commit that referenced this pull request Dec 19, 2023
PR-URL: #50486
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants