Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

feat: get Filecoin.ClientRetrieve to actually save a file and update IPFS #791

Merged
merged 22 commits into from Mar 4, 2021
Merged
Show file tree
Hide file tree
Changes from 13 commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
a74c4d0
get Filecoin.ClientRetrieve to actually save a file
mikeseese Jan 13, 2021
c1d3fe6
save to disk in chunks rather than loading the whole file into heap
mikeseese Jan 16, 2021
2c8b097
update ipfs server to have a more generic options setup
mikeseese Jan 15, 2021
21bf9e7
only run filecoin tests on node v12+
mikeseese Jan 23, 2021
8ff7a56
get tests to work
mikeseese Jan 23, 2021
bf113eb
update ClientRetrieve test to check downloaded content
mikeseese Jan 25, 2021
a33aea0
add api docs link for FileRef
mikeseese Jan 25, 2021
a2cb6b9
get tests to run after rebase
mikeseese Jan 29, 2021
53702e1
support strictNullCheck and noImplicitAny
mikeseese Jan 29, 2021
597b17e
make sure we run test.filecoin (issue with rebase)
mikeseese Feb 4, 2021
339f2ee
update type declarations
mikeseese Feb 4, 2021
32808cf
remove type C from filecoin things
mikeseese Feb 5, 2021
c502fd3
update type declarations
mikeseese Feb 5, 2021
771d476
check for not node versions 10/11 vs for 12+
mikeseese Feb 22, 2021
cdb1df1
only fetch object size if it wasn't provided in proposal
mikeseese Feb 22, 2021
d08baa5
address code review concerns for during client retrieve for large files
mikeseese Feb 22, 2021
95fee05
change filecoin scripts to not run if node12 req not met
mikeseese Feb 24, 2021
701fb84
add a `finally` block to ensure we close the file writestream
mikeseese Feb 24, 2021
728d701
set engine-strict=false to allow installing filecoin in dev env
mikeseese Feb 24, 2021
bdb2e13
remove test.filecoin from github workflows
mikeseese Feb 24, 2021
9ac3a61
rename skip script
mikeseese Feb 24, 2021
3e3ce66
update usage comment with new filename
mikeseese Feb 24, 2021
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
4 changes: 4 additions & 0 deletions .github/workflows/pr.yml
Expand Up @@ -52,3 +52,7 @@ jobs:
- run: npm test
env:
FORCE_COLOR: 1
- run: npm run test.filecoin
if: ${{ startsWith(matrix.node, '12.') || startsWith(matrix.node, '13.') || startsWith(matrix.node, '14.') }}
env:
FORCE_COLOR: 1
8 changes: 8 additions & 0 deletions .github/workflows/push.yml
Expand Up @@ -26,6 +26,10 @@ jobs:
- run: npm test
env:
FORCE_COLOR: 1
- run: npm run test.filecoin
if: ${{ startsWith(matrix.node, '12.') || startsWith(matrix.node, '13.') || startsWith(matrix.node, '14.') }}
mikeseese marked this conversation as resolved.
Show resolved Hide resolved
env:
FORCE_COLOR: 1

build-all:
strategy:
Expand Down Expand Up @@ -71,3 +75,7 @@ jobs:
- run: npm test
env:
FORCE_COLOR: 1
- run: npm run test.filecoin
if: ${{ startsWith(matrix.node, '12.') || startsWith(matrix.node, '13.') || startsWith(matrix.node, '14.') }}
env:
FORCE_COLOR: 1
3 changes: 2 additions & 1 deletion package.json
Expand Up @@ -15,7 +15,8 @@
"postinstall": "lerna bootstrap && npx patch-package && ts-node scripts/link-ts-references.ts && npm run tsc && ts-node scripts/postinstall",
"reinstall": "npm run clean && npm install",
"start": "lerna exec --loglevel=silent --scope ganache -- npm run start --silent -- ",
"test": "lerna exec -- npm run test",
"test": "lerna exec --ignore @ganache/filecoin* -- npm run test",
"test.filecoin": "npx lerna exec --scope @ganache/filecoin* -- npm run test",
mikeseese marked this conversation as resolved.
Show resolved Hide resolved
"tsc": "ttsc --build src",
"tsc.clean": "npx lerna exec -- npx shx rm -rf lib dist",
"tsc.filecoin.declarations": "lerna run tsc.declarations --scope @ganache/filecoin && pretty-quick src/chains/filecoin/types"
Expand Down