fs: remove unused assignment by Trott · Pull Request #35882 · nodejs/node
-
-
Notifications
You must be signed in to change notification settings - Fork 35.7k
Merged
Merged
Conversation
Trott
commented
Oct 30, 2020
Trott
commented
Member
Checklist
-
make -j4 test(UNIX), orvcbuild test(Windows) passes - commit message follows commit guidelines
nodejs-github-bot
added
the
fs
label
cjihrig
approved these changes
github-actions
Bot
removed
the
request-ci
label
nodejs-github-bot
commented
Oct 30, 2020
nodejs-github-bot commented
Collaborator
nodejs-github-bot
commented
Oct 30, 2020
nodejs-github-bot commented
Collaborator
watilde
approved these changes
PR-URL: nodejs#35882 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Trott
force-pushed
the
unused-options
branch
from
c50830c to
1d525f5
Compare
Trott
commented
Nov 7, 2020
Trott commented
Member Author
Landed in 1d525f5
Trott
merged commit
1d525f5
into
nodejs:master
Trott
deleted the
unused-options
branch
danielleadams pushed a commit that referenced this pull request
PR-URL: #35882 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
danielleadams
mentioned this pull request
Merged
targos
added
the
dont-land-on-v14.x
label
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment