Skip to content

deps: update ada to 2.8.0 - #53254

Merged
nodejs-github-bot merged 1 commit into
mainfrom
actions/tools-update-ada
Jun 4, 2024
Merged

deps: update ada to 2.8.0#53254
nodejs-github-bot merged 1 commit into
mainfrom
actions/tools-update-ada

Conversation

@nodejs-github-bot

@nodejs-github-botnodejs-github-bot commented Jun 2, 2024

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot added the dependencies Pull requests that update a dependency file. label Jun 2, 2024
@nodejs-github-bot

nodejs-github-bot commented Jun 2, 2024

Copy link
Copy Markdown
CollaboratorAuthor

Review requested:

  • @nodejs/security-wg
  • @nodejs/url

@nodejs-github-botnodejs-github-bot added the needs-ci PRs that need a full CI run. label Jun 2, 2024
@anonrig

Copy link
Copy Markdown
Member

nit: we have done some performance improvements on Ada. @nodejs/performance can anybody run the benchmark ci on URL?

@anonriganonrig added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 2, 2024
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 2, 2024
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

Copy link
Copy Markdown
CollaboratorAuthor

@aduh95aduh95 added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 2, 2024
@jasnelljasnell added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 2, 2024
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 4, 2024
@nodejs-github-bot
nodejs-github-bot merged commit 0986817 into mainJun 4, 2024
@nodejs-github-bot

Copy link
Copy Markdown
CollaboratorAuthor

Landed in 0986817

@nodejs-github-bot
nodejs-github-bot deleted the actions/tools-update-ada branch June 4, 2024 00:33
RafaelGSS pushed a commit that referenced this pull request Jun 7, 2024
PR-URL: #53254
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@RafaelGSSRafaelGSS mentioned this pull request Jun 7, 2024
eliphazbouye pushed a commit to eliphazbouye/node that referenced this pull request Jun 20, 2024
PR-URL: nodejs#53254
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
marco-ippolito pushed a commit that referenced this pull request Jul 19, 2024
PR-URL: #53254
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@marco-ippolitomarco-ippolito mentioned this pull request Jul 19, 2024
aduh95 pushed a commit that referenced this pull request Nov 3, 2024
PR-URL: #53254
Reviewed-By: Yagiz Nizipli <yagiz.nizipli@sentry.io>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
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.dependenciesPull requests that update a dependency file.needs-ciPRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@nodejs-github-bot@anonrig@jasnell@marco-ippolito@aduh95