Skip to content

Commit

Permalink
fix!: update deps (#30)
Browse files Browse the repository at this point in the history
BREAKING CHANGE: make unixfs importer on ipfs-car use same defaults as lotus (web3-storage/web3.storage#170)
  • Loading branch information
vasco-santos committed Jan 5, 2022
1 parent 863f23f commit 3a655ff
Show file tree
Hide file tree
Showing 9 changed files with 5,675 additions and 7,180 deletions.
2 changes: 0 additions & 2 deletions .dockerignore

This file was deleted.

5 changes: 0 additions & 5 deletions Dockerfile

This file was deleted.

6 changes: 2 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
## Example usage

```yaml
uses: web3-storage/add-to-web3@v1
uses: web3-storage/add-to-web3@v2
id: web3
with:
web3_token: ${{ secrets.WEB3_STORAGE_TOKEN }}
Expand Down Expand Up @@ -64,8 +64,6 @@ e.g. `https://dweb.link/ipfs/bafkreicysg23kiwv34eg2d7qweipxwosdo2py4ldv42nbaugul

💌 Considerate contributions welcome!

*Of note* This is supposed to be a Javascript flavour GitHub Action, but the JS runner is [stuck on node12](https://github.com/actions/runner/issues/772v), and we need at least node14. Until the glorious future where the current node version is supported, we wrap the action in a container.

The `dist` folder is commited to the repo as is the curious cultural norm with JS actions, as the repo is the delivery mechanism, so to spare some cycles for the user users, all the deps are bundled into a single /dist/index.js monolith. This no longer makes much sense as we're also wrapping it in a container, but the dream is that the new node16 runner lands, and we can just delete the Dockers and move on.
The `dist` folder is commited to the repo as is the curious cultural norm with JS actions, as the repo is the delivery mechanism, so to spare some cycles for the user users, all the deps are bundled into a single /dist/index.js monolith.

<h3 align="center"><a href="https://web3.storage">⁂</a></h3>
8 changes: 2 additions & 6 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,5 @@ outputs:
cid:
description: 'The IPFS Content ID for the directory'
runs:
# TODO: we need node14.14 minimum.
# https://github.com/actions/runner/issues/772
# using: 'node12'
# main: 'dist/index.js'
using: 'docker'
image: 'Dockerfile'
using: 'node16'
main: 'dist/index.js'
Loading

0 comments on commit 3a655ff

Please sign in to comment.