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.

Sign up for GitHub

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

Jump to bottom

test: remove outdated documentation #22009

Closed
wants to merge 1 commit into from
Closed

test: remove outdated documentation #22009

wants to merge 1 commit into from

Conversation

maclover7
Copy link
Contributor

As noted by @richardlau, Makefile and vcbuild.bat no longer need to
be updated since they run the default suite.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

@maclover7
test: remove outdated documentation
eafc13d 
As noted by @richardlau, `Makefile` and `vcbuild.bat` no longer need to
be updated since they run the `default` suite.
@nodejs-github-bot
Copy link
Collaborator

@maclover7 build started: https://ci.nodejs.org/blue/organizations/jenkins/node-test-pull-request-lite-pipeline/detail/node-test-pull-request-lite-pipeline/371/pipeline

@nodejs-github-bot nodejs-github-bot added the test Issues and PRs related to the tests. label Jul 28, 2018
@Trott

This comment has been minimized.

Sign in to view
@maclover7
Copy link
Contributor Author

Instead of removing the text, should it be updated to say to consider whether to update the default suite in test.py?

The default suite is "opt out" (tools/test.py lists all directories and then removes all "ignored" ones). When adding a new test directory, tools/test.py should be able to run without any updated configuration

@BridgeAR BridgeAR added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jul 29, 2018
@maclover7
Copy link
Contributor Author

Landed in 48e1763

@maclover7 maclover7 closed this Jul 31, 2018
@maclover7 maclover7 deleted the jm-rm-testdoc branch July 31, 2018 03:47
maclover7 added a commit that referenced this pull request Jul 31, 2018
@maclover7
test: remove outdated documentation
48e1763 
As noted by @richardlau, `Makefile` and `vcbuild.bat` no longer need to
be updated since they run the `default` suite.

PR-URL: #22009
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@targos targos added backport-requested-v10.x and removed author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels Jul 31, 2018
@targos
Copy link
Member

targos commented Jul 31, 2018

Depends on #21910 to land on v10.x-staging

targos pushed a commit that referenced this pull request Aug 2, 2018
@maclover7 @targos
test: remove outdated documentation
e2d97ee 
As noted by @richardlau, `Makefile` and `vcbuild.bat` no longer need to
be updated since they run the `default` suite.

PR-URL: #22009
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@rvagg rvagg mentioned this pull request Aug 13, 2018
Merged
This was referenced Aug 16, 2018
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
@snyk-bot snyk-bot mentioned this pull request Apr 1, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BridgeAR BridgeAR BridgeAR approved these changes

@richardlau richardlau richardlau approved these changes

@lpinca lpinca lpinca approved these changes

@jasnell jasnell jasnell approved these changes

Assignees
No one assigned
Labels
test Issues and PRs related to the tests.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@maclover7 @nodejs-github-bot @Trott @targos @jasnell @lpinca @richardlau @BridgeAR