Skip to content

test: remove unused symlink loop#31267

Closed
cjihrig wants to merge 1 commit intonodejs:masterfrom
cjihrig:symloop
Closed

test: remove unused symlink loop#31267
cjihrig wants to merge 1 commit intonodejs:masterfrom
cjihrig:symloop

Conversation

@cjihrig
Copy link
Copy Markdown
Contributor

@cjihrig cjihrig commented Jan 9, 2020

This symlink loop was originally intended to be accessed from a test. Instead, a loop is created in the test itself, but this was unfortunately forgotten about. It's unused, so remove it.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines

👍 here if you'd like to fast track this.

Loading
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. fast-track PRs that do not need to wait for 72 hours to land. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants