Skip to content

Commit

Permalink
fix: remove remaining references to packages/wallet (#253)
Browse files Browse the repository at this point in the history
Motivation:
* latest commit on main branch was failing release workflow
[here](https://github.com/web3-storage/w3protocol/actions/runs/3621669639/jobs/6105393187#step:2:35)
* I think it's because the release-please config/manifest still referred
to `package/wallet`, which this PR removes
* seems like logical followup to
#248

Co-authored-by: Hugo Dias <hugomrdias@gmail.com>
  • Loading branch information
gobengo and hugomrdias authored Dec 6, 2022
1 parent 2af1b76 commit 61b5227
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 34 deletions.
3 changes: 1 addition & 2 deletions .github/release-please-config.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
"packages/access-client": {},
"packages/access-api": {},
"packages/capabilities": {},
"packages/upload-client": {},
"packages/wallet": {}
"packages/upload-client": {}
}
}
1 change: 0 additions & 1 deletion .github/release-please-manifest.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
{
"packages/wallet": "1.0.0",
"packages/access-client": "7.0.2",
"packages/access-api": "3.0.0",
"packages/capabilities": "1.0.0",
Expand Down
25 changes: 0 additions & 25 deletions .github/workflows/manual.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ on:
- access-api
- upload-client
- access-client
- wallet
- docs
environment:
description: 'Environment to deploy'
Expand All @@ -23,30 +22,6 @@ on:
- staging
- dev
jobs:
deploy-wallet:
runs-on: ubuntu-latest
if: github.event.inputs.package == 'wallet'
steps:
- uses: actions/checkout@v3
- uses: pnpm/action-setup@v2.2.3
with:
version: 7
- uses: actions/setup-node@v3
with:
node-version: 16
cache: 'pnpm'
- uses: actions/cache@v2
with:
path: ${{ github.workspace }}/packages/wallet/.next/cache
key: ${{ runner.os }}-nextjs-${{ hashFiles('**/pnpm-lock.yaml') }}-${{ hashFiles('packages/wallet/**.[jt]s', 'packages/wallet/**.[jt]sx') }}
restore-keys: |
${{ runner.os }}-nextjs-${{ hashFiles('**/pnpm-lock.yaml') }}-
- run: pnpm install
- run: pnpm -r --filter wallet run build
- run: pnpm exec wrangler pages publish --project-name ucan-protocol --branch main ./packages/wallet/out
env:
CLOUDFLARE_API_TOKEN: ${{ secrets.CF_TOKEN }}
CLOUDFLARE_ACCOUNT_ID: ${{ secrets.CF_ACCOUNT_ID }}
deploy-access-api:
if: github.event.inputs.package == 'access-api'
uses: './.github/workflows/reusable-deploy-api.yml'
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,6 @@ jobs:
manifest-file: .github/release-please-manifest.json
default-branch: main
release-type: node
pages:
needs: release
if: contains(fromJson(needs.release.outputs.paths_released), 'packages/wallet')
runs-on: ubuntu-latest
steps:
- run: echo 'pages'
access-api:
needs: release
if: contains(fromJson(needs.release.outputs.paths_released), 'packages/access-api')
Expand Down

0 comments on commit 61b5227

Please sign in to comment.