Skip to content

Conversation

@privatenumber
Copy link
Contributor

@privatenumberprivatenumber commented Dec 4, 2023

The parentURL property was missing in options documentation

Development

Ran make test-doc to lint docs

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/loaders

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. module Issues and PRs related to the module subsystem. labels Dec 4, 2023
@GeoffreyBooth
Copy link
Member

Can you please confirm that it is supported there? I remember us debating whether to support parentURL as the second argument or within options or both; I know it’s supported as the second argument, is it also supported in options?

If it’s supported in both, is that what we want? What happens if you define it in both places?

@JakobJingleheimer
Copy link
Member

Can you please confirm that it is supported there? I remember us debating whether to support parentURL as the second argument or within options or both; I know it’s supported as the second argument, is it also supported in options?

If it’s supported in both, is that what we want? What happens if you define it in both places?

This was answered in #51039 (comment)

@nodejs-github-bot
Copy link
Collaborator

@GeoffreyBoothGeoffreyBooth added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Dec 5, 2023
@aduh95aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 5, 2023
@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 Dec 6, 2023
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/51039 ✔ Done loading data for nodejs/node/pull/51039 ----------------------------------- PR info ------------------------------------ Title module: document `parentURL` in register options (#51039) ⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile! Branch privatenumber:module-docs-parentURL -> nodejs:main Labels doc, module, author ready Commits 3 - module: document parentURL in register options - Merge branch 'main' into module-docs-parentURL - Update doc/api/module.md Committers 2 - Hiroki Osame - GitHub PR-URL: https://github.com/nodejs/node/pull/51039 Reviewed-By: Antoine du Hamel Reviewed-By: Luigi Pinca Reviewed-By: Jacob Smith Reviewed-By: Geoffrey Booth ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/51039 Reviewed-By: Antoine du Hamel Reviewed-By: Luigi Pinca Reviewed-By: Jacob Smith Reviewed-By: Geoffrey Booth -------------------------------------------------------------------------------- ℹ This PR was created on Mon, 04 Dec 2023 05:08:14 GMT ✔ Approvals: 4 ✔ - Antoine du Hamel (@aduh95) (TSC): https://github.com/nodejs/node/pull/51039#pullrequestreview-1763062424 ✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/51039#pullrequestreview-1763255497 ✔ - Jacob Smith (@JakobJingleheimer): https://github.com/nodejs/node/pull/51039#pullrequestreview-1763362015 ✔ - Geoffrey Booth (@GeoffreyBooth) (TSC): https://github.com/nodejs/node/pull/51039#pullrequestreview-1763999641 ✔ 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 51039 From https://github.com/nodejs/node * branch refs/pull/51039/merge -> FETCH_HEAD ✔ Fetched commits as ac9e594e3224..243207ade200 -------------------------------------------------------------------------------- error: commit 042ae1fc72072eaa78b12f928629593efbeba626 is a merge but no -m option was given. fatal: cherry-pick failed [main e2151c7755] module: document parentURL in register options Author: Hiroki Osame Date: Mon Dec 4 14:06:04 2023 +0900 1 file changed, 4 insertions(+) ✘ Failed to apply patches
https://github.com/nodejs/node/actions/runs/7110494786

@aduh95aduh95 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 Dec 6, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 6, 2023
@nodejs-github-botnodejs-github-bot merged commit 342ddb0 into nodejs:mainDec 6, 2023
@nodejs-github-bot
Copy link
Collaborator

Landed in 342ddb0

RafaelGSS pushed a commit that referenced this pull request Dec 15, 2023
PR-URL: #51039 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Jacob Smith <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]>
@RafaelGSSRafaelGSS mentioned this pull request Dec 15, 2023
richardlau pushed a commit that referenced this pull request Mar 25, 2024
PR-URL: #51039 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Jacob Smith <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]>
@richardlaurichardlau mentioned this pull request Mar 25, 2024
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.docIssues and PRs related to the documentations.moduleIssues and PRs related to the module subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@privatenumber@nodejs-github-bot@GeoffreyBooth@JakobJingleheimer@lpinca@aduh95