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

Commit

Permalink
fix: wasm error during transaction (#244)
Browse files Browse the repository at this point in the history
Closes #243 
- polkadot libraries were updated
- updates for Metamask cli and build 
- switched to node 18 (16 got deprecated with the new metamask cli)
- GitHub actions uses v 18 of node 
- Removed shasum check on GitHub actions as a new CLI updates the
manifest file if it doesn't match
  • Loading branch information
Lykhoyda authored Aug 27, 2024
1 parent ea149fb commit c2f51e5
Show file tree
Hide file tree
Showing 15 changed files with 3,557 additions and 1,128 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/cd.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,14 +24,14 @@ jobs:
monorepo-tags: true
changelog-types: '[{"type":"feat","section":"Features","hidden":false},{"type":"fix","section":"Bug Fixes","hidden":false},{"type":"chore","section":"Miscellaneous","hidden":false}]'

- uses: actions/checkout@v3
- uses: actions/checkout@v4
# these if statements ensure that a publication only occurs when
# a new release is created:
if: ${{ steps.release.outputs.releases_created }}
- uses: actions/setup-node@v3

- uses: actions/setup-node@v4
with:
node-version: 16
node-version: 18
registry-url: 'https://registry.npmjs.org'
if: ${{ steps.release.outputs.releases_created }}

Expand Down
20 changes: 4 additions & 16 deletions .github/workflows/ci-workflow.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,28 +15,16 @@ jobs:
# This job runs on Linux
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: '16.x'
node-version: 18
- run: corepack enable
- run: yarn install --immutable
- name: Build
run: yarn run build
- name: Run linter
run: yarn run lint
- uses: tj-actions/verify-changed-files@v17
if: github.ref_name != 'master'
id: verify-changed-files
with:
files: |
packages/snap/snap.manifest.json
- name: Check if snap manifest updated
if: steps.verify-changed-files.outputs.files_changed == 'true'
run: |
echo "Snap manifest not updated: ${{ steps.verify-changed-files.outputs.changed_files }}
or @metamask/snaps-cli version is different from local version"
exit 1
- name: Run tests
run: yarn run test

Expand All @@ -48,7 +36,7 @@ jobs:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
with:
node-version: '16.x'
node-version: 18
- run: corepack enable
- name: Install
run: yarn install --immutable
Expand Down
2 changes: 1 addition & 1 deletion .nvmrc
Original file line number Diff line number Diff line change
@@ -1 +1 @@
16
18
6 changes: 3 additions & 3 deletions packages/adapter/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,9 @@
"access": "public"
},
"dependencies": {
"@polkadot/api": "^10.9.1",
"@polkadot/extension-inject": "^0.46.5",
"@polkadot/types-augment": "^10.9.1"
"@polkadot/api": "^12.4.2",
"@polkadot/extension-inject": "^0.52.3",
"@polkadot/types-augment": "^12.4.2"
},
"scripts": {
"prebuild": "rm -rf build",
Expand Down
2 changes: 1 addition & 1 deletion packages/adapter/tsconfig.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"compilerOptions": {
"target": "es6",
"target": "ES2020",
"outDir": "build",
"rootDir": "src",
"moduleResolution": "node",
Expand Down
4 changes: 3 additions & 1 deletion packages/example/.eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@ module.exports = {
},
rules: {
'@typescript-eslint/no-unsafe-return': 'warn',
"@typescript-eslint/no-unsafe-call": 'warn',
'@typescript-eslint/no-unsafe-assignment': 'warn',
'@typescript-eslint/no-misused-promises': 'off'
'@typescript-eslint/no-misused-promises': 'off',
'@typescript-eslint/no-unsafe-member-access': 'warn',
}
};
Loading

0 comments on commit c2f51e5

Please sign in to comment.