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

fixed utilization of updated version of tar, added unit tests #45

Merged
merged 10 commits into from
Jun 22, 2019
Merged

fixed utilization of updated version of tar, added unit tests #45

Changes from 3 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
2df4bd5
Merge pull request #1 from sutt0n/update-tar
sutt0n May 14, 2019
ea73040
extract is no longer used
codyseibert May 14, 2019
f09422d
Merge pull request #2 from codyseibert/latest_tar_support
sutt0n May 14, 2019
a255e6a
Update installer.js
codyseibert May 14, 2019
0d28221
Merge pull request #3 from codyseibert/patch-1
sutt0n May 14, 2019
af5fca3
added unit tests
sutt0n May 14, 2019
7db7d79
added unit test file
sutt0n May 14, 2019
4044cad
Merge branch 'master' of https://github.com/sutt0n/dynamodb-localhost…
sutt0n May 14, 2019
ca7e8c8
fixed unit tests
sutt0n May 14, 2019
1aebdce
Merge pull request #4 from sutt0n/unit-tests
sutt0n May 14, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion dynamodb/installer.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ var download = function (downloadUrl, installPath, callback) {

response
.pipe(zlib.Unzip())
.pipe(tar.Extract({
.pipe(new tar.Unpack({
path: installPath
}))
.on('data', function (chunk) {
Expand Down