lib,tools: use `internal/url`, restrict `url` in internal code by LiviaMedeiros · Pull Request #49590

lib,tools: use `internal/url`, restrict `url` in internal code by LiviaMedeiros · Pull Request #49590 · nodejs/node

added debugger

Issues and PRs related to the debugger subsystem.

lib / src

Issues and PRs related to general changes in the lib or src directory.

needs-ci

PRs that need a full CI run.

test_runner

Issues and PRs related to the test runner subsystem.

labels

Sep 10, 2023

LiviaMedeiros changed the title ~~lib: restrict internal code from using public url module~~ lib,tools: use internal/url, restrict url in internal code

Sep 10, 2023

LiviaMedeiros added the author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

label

Sep 11, 2023

LiviaMedeiros removed the author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

label

Sep 11, 2023

LiviaMedeiros added the author ready

PRs that have at least one approval, no pending requests for changes, and a CI started.

label

Sep 11, 2023

aduh95 added the commit-queue

Add this label to land a pull request using GitHub Actions.

label

Sep 13, 2023

nodejs-github-bot pushed a commit that referenced this pull request

Sep 13, 2023

PR-URL: #49590
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

nodejs-github-bot pushed a commit that referenced this pull request

Sep 13, 2023

PR-URL: #49590
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

ruyadorno pushed a commit that referenced this pull request

Sep 28, 2023

PR-URL: #49590
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

ruyadorno pushed a commit that referenced this pull request

Sep 28, 2023

PR-URL: #49590
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

This was referenced

Sep 28, 2023

alexfernandez pushed a commit to alexfernandez/node that referenced this pull request

Nov 1, 2023

PR-URL: nodejs#49590
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

alexfernandez pushed a commit to alexfernandez/node that referenced this pull request

Nov 1, 2023

PR-URL: nodejs#49590
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>