Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Remove wrench, fs-plus in favor of fs-extra #836

Closed
wants to merge 36 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
ad6abf9
Step 0: install fs-extra
50Wliu Apr 22, 2019
307c313
Step 1: use fs-extra in fs.coffee
50Wliu Apr 22, 2019
50c266b
Step 2: change all the things?
50Wliu Apr 22, 2019
a270ce7
Ugh
50Wliu Apr 22, 2019
4229775
More fixes
50Wliu Apr 23, 2019
ae36c63
Get rid of the remaining fs-plus functions
50Wliu Apr 23, 2019
fe6eac4
Fix initializing packages
50Wliu Apr 23, 2019
88ef0f5
That took longer than expected
50Wliu Apr 23, 2019
8c446df
Add missing mkdirps to upgrade-spec
50Wliu Apr 23, 2019
ae7c496
:fire: wrench
50Wliu Apr 23, 2019
cfb0c47
:fire: fs-plus, mv, ncp
50Wliu Apr 23, 2019
5e61195
Oops
50Wliu Apr 23, 2019
0f2fe0c
Don't know how I missed those
50Wliu Apr 23, 2019
fd56c14
copySync
50Wliu Apr 23, 2019
d258284
Debug it
50Wliu Apr 24, 2019
e633ef8
More debug
50Wliu Apr 24, 2019
c292527
Ah hah
50Wliu Apr 24, 2019
19d54d4
Bam, got it!
50Wliu Apr 24, 2019
cab9db2
Silence output
50Wliu Apr 24, 2019
88b6c20
Is it the link specs?
50Wliu Apr 24, 2019
e24d0da
Nope - somewhere in install-spec?
50Wliu Apr 24, 2019
c4664ac
Ah, yes
50Wliu Apr 24, 2019
f1330b8
Wow it really was just silencing the output
50Wliu Apr 25, 2019
f860435
If statSync triggers an exception, fallback to 32-bit Git
50Wliu Apr 25, 2019
51bc62a
Ensure no behavior changes
50Wliu Apr 25, 2019
0b71652
Merge branch 'master' into wl-fs-extra
50Wliu Apr 26, 2019
207531a
Wah
50Wliu Apr 27, 2019
c83d66d
Merge branch 'master' into wl-fs-extra
50Wliu Apr 27, 2019
e7c293f
Where is this node-gyp??
50Wliu Apr 27, 2019
1e9ad02
What about this
50Wliu Apr 27, 2019
675734c
Verbose it
50Wliu Apr 27, 2019
3e1d4ee
Wrong function...
50Wliu Apr 27, 2019
81547a9
Fix the bundle issues???
50Wliu Apr 27, 2019
8effbd9
Undo those require.resolve changes
50Wliu Apr 27, 2019
44b7567
Forgot the silenceOutput change
50Wliu Apr 27, 2019
3984f4c
Merge branch 'master' into wl-fs-extra
50Wliu May 8, 2019
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
180 changes: 84 additions & 96 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

7 changes: 3 additions & 4 deletions package.json
Expand Up @@ -26,13 +26,13 @@
"asar-require": "0.3.0",
"async": "~0.2.8",
"colors": "~0.6.1",
"expand-tilde": "^2.0.2",
"first-mate": "6.2.0",
"fs-plus": "2.x",
"fs-extra": "^7.0.1",
"git-utils": "^4.0",
"hosted-git-info": "^2.1.4",
"keytar": "^4.0",
"mv": "2.0.0",
"ncp": "~0.5.1",
"klaw-sync": "^6.0.0",
"npm": "6.2.0",
"open": "0.0.5",
"q": "~0.9.7",
Expand All @@ -45,7 +45,6 @@
"temp": "^0.8.3",
"underscore-plus": "1.x",
"wordwrap": "0.0.2",
"wrench": "~1.5.1",
"yargs": "^3.23.0"
},
"devDependencies": {
Expand Down
5 changes: 2 additions & 3 deletions script/download-node.js
@@ -1,5 +1,4 @@
var fs = require('fs');
var mv = require('mv');
var fs = require('fs-extra');
var zlib = require('zlib');
var path = require('path');

Expand Down Expand Up @@ -56,7 +55,7 @@ var copyNodeBinToLocation = function(callback, version, targetFilename, fromDire
var arch = identifyArch();
var subDir = "node-" + version + "-" + process.platform + "-" + arch;
var downloadedNodePath = path.join(fromDirectory, subDir, 'bin', 'node');
return mv(downloadedNodePath, targetFilename, {mkdirp: true}, function(err) {
fs.move(downloadedNodePath, targetFilename, function(err) {
if (err) {
callback(err);
return;
Expand Down
2 changes: 1 addition & 1 deletion spec/apm-cli-spec.coffee
@@ -1,6 +1,6 @@
fs = require 'fs-extra'
path = require 'path'
temp = require 'temp'
fs = require 'fs'
apm = require '../lib/apm-cli'

describe 'apm command line interface', ->
Expand Down