Skip to content

doc: enrich test commands with executable #44347

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

Merged
merged 10 commits into from
Feb 6, 2023

Conversation

tony-go
Copy link
Member

@tony-go tony-go commented Aug 22, 2022

Hey 👋

A tiny pull request to enrich test commands in the documentation.

cc @arkerone

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. labels Aug 22, 2022
@tony-go tony-go marked this pull request as ready for review August 23, 2022 15:16

Verified

This commit was signed with the committer’s verified signature.
targos Michaël Zasso

Verified

This commit was signed with the committer’s verified signature.
targos Michaël Zasso
@tony-go tony-go requested a review from aduh95 August 24, 2022 06:38

Verified

This commit was signed with the committer’s verified signature.
targos Michaël Zasso
@tony-go tony-go requested review from aduh95 and RaisinTen and removed request for aduh95 and RaisinTen August 24, 2022 16:53
Copy link
Member

@RaisinTen RaisinTen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, would be nice if you could also include the reasoning (#44347 (comment)) in the description

Copy link
Member

@RaisinTen RaisinTen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After #44407 lands, tools/test.py should work without explicitly writing out the command name. Calling it like python ... would only be required on Windows.

Verified

This commit was signed with the committer’s verified signature.
targos Michaël Zasso
@tony-go
Copy link
Member Author

tony-go commented Feb 4, 2023

@RaisinTen I finally updated the doc, could PTAL ^^

@tony-go tony-go requested review from RaisinTen and removed request for aduh95 February 4, 2023 11:36

Verified

This commit was signed with the committer’s verified signature.
targos Michaël Zasso
This reverts commit d6ee7d8.

Verified

This commit was signed with the committer’s verified signature.
targos Michaël Zasso
@tony-go tony-go requested review from richardlau and removed request for RaisinTen February 5, 2023 15:32
@tony-go
Copy link
Member Author

tony-go commented Feb 5, 2023

Finally, remove the python exec from examples and add a note for windows.

Copy link
Member

@RaisinTen RaisinTen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@RaisinTen RaisinTen added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Feb 6, 2023
@RaisinTen RaisinTen added commit-queue Add this label to land a pull request using GitHub Actions. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. labels Feb 6, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Feb 6, 2023
@nodejs-github-bot nodejs-github-bot merged commit 2dacd07 into nodejs:main Feb 6, 2023
@nodejs-github-bot
Copy link
Collaborator

Landed in 2dacd07

MylesBorins pushed a commit that referenced this pull request Feb 18, 2023

Verified

This commit was signed with the committer’s verified signature. The key has expired.
danielleadams Danielle Adams
PR-URL: #44347
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
@MylesBorins MylesBorins mentioned this pull request Feb 19, 2023
danielleadams pushed a commit that referenced this pull request Apr 11, 2023

Verified

This commit was signed with the committer’s verified signature. The key has expired.
danielleadams Danielle Adams
PR-URL: #44347
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. build Issues and PRs related to build files or the CI. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants