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.

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

feat(erc20): Single Token Representation V2 #2517

Draft
wants to merge 46 commits into
base: main
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
46 commits
Select commit Hold shift + click to select a range
f00501a
placeholder
Vvaradinov Jan 11, 2024
1689dfe
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
Vvaradinov Jan 15, 2024
267e4f5
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
Vvaradinov Jan 17, 2024
bd10bcd
fix(str): Revert deleted RegisterCoinProposal protos and types (#2276)
Vvaradinov Jan 18, 2024
2ebc54a
chore(str): Remove MsgConvertCoin and all it's related protos, types …
Vvaradinov Jan 18, 2024
2e5c03d
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
Vvaradinov Jan 19, 2024
909ed62
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
Vvaradinov Jan 22, 2024
4f530cc
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
MalteHerrmann Jan 25, 2024
3950bee
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
MalteHerrmann Jan 30, 2024
d6101d5
imp(str): Updated ERC20 IBC middleware logic (#2159)
Vvaradinov Jan 31, 2024
eb4e4fc
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
Vvaradinov Jan 31, 2024
0a27c2a
imp(upgrade): Add migration logic for STR v2 to upgrade handler (#2290)
MalteHerrmann Feb 8, 2024
ea7ded7
tests(str): Add native coins ibc integration tests (#2327)
ramacarlucho Feb 8, 2024
f808bd3
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
Vvaradinov Feb 8, 2024
28dbbcb
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
MalteHerrmann Feb 12, 2024
eff5480
imp(erc20-precompile): Adjust required gas on ERC20 extension (#2142)
MalteHerrmann Feb 12, 2024
5d03a5e
feat(str): Disable `ToggleConversion` for Native Coins pairs (#2340)
ramacarlucho Feb 13, 2024
f078f08
imp(tests): Move ERC-20 balance to utils (#2347)
MalteHerrmann Feb 13, 2024
fee60bb
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
MalteHerrmann Feb 13, 2024
e437db7
fix(evm): Add DynamicPrecompiles params (#2339)
facs95 Feb 21, 2024
2545bf5
str(tests): Add nix e2e tests (#2313)
Vvaradinov Feb 21, 2024
f71fc96
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
facs95 Feb 21, 2024
f5575b5
test(str-v2): Handle tokenfactory edge case and nix test (#2350)
Vvaradinov Feb 21, 2024
23c47a6
fix(str-v2): add str-v2 nix tests to CI and remove old Outpost nix CI…
Vvaradinov Feb 22, 2024
802b9f9
fix: add missing key
Vvaradinov Feb 22, 2024
4e87a66
format and lint
Vvaradinov Feb 22, 2024
810a1fc
fix: remove outpost nix tests
Vvaradinov Feb 22, 2024
5bdf8ce
imp(evm): Add ApplyTransaction benchmark with new setup (#2372)
facs95 Feb 26, 2024
98db7ae
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
Vvaradinov Feb 26, 2024
f36c499
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
Vvaradinov Feb 26, 2024
383b4cd
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
MalteHerrmann Feb 27, 2024
a0f506c
imp(str-v2): Add Event after successful ERC20 extension registration …
Vvaradinov Mar 20, 2024
e8fe848
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch
Vvaradinov Mar 20, 2024
ffd1b7f
fix: changelog
Vvaradinov Mar 20, 2024
c2d9bb2
fix: remove unused files from merge
Vvaradinov Mar 20, 2024
3117a73
fix: add check for bank balance to ibc test
Vvaradinov Mar 20, 2024
70559b7
fix: add correct denom
Vvaradinov Mar 20, 2024
96a526d
fix: re-add ERC20Keeper
Vvaradinov Mar 20, 2024
cf65018
run make format
Vvaradinov Mar 20, 2024
068f95c
fix: format and lint and remove Revenue reference
Vvaradinov Mar 20, 2024
f04333a
Merge remote-tracking branch 'origin/Vvaradinov/feat-strv2-feature-br…
Vvaradinov Mar 20, 2024
6d5d101
fix(str): Apply changes from internal audit (#2476)
ramacarlucho Apr 12, 2024
63c73fd
imp(evm): Add `PreExecuteCallback` function to EVM for precompiles in…
facs95 Apr 12, 2024
1717667
fix(str): Add some changes from the internal STRv2 audit (#2443)
MalteHerrmann Apr 16, 2024
744b59a
Merge branch 'main' into Vvaradinov/feat-strv2-feature-branch (#2496)
facs95 Apr 22, 2024
3722159
chore(strv2): Merge main back into feature branch (#2518)
facs95 Apr 30, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
4 changes: 3 additions & 1 deletion .github/workflows/ante-benchmark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ on:
branches:
- main

permissions: read-all

jobs:
ante-benchmark-test:
runs-on: ubuntu-latest
Expand All @@ -19,7 +21,7 @@ jobs:
- name: Setup Golang
uses: actions/setup-go@v5
with:
go-version: '1.21'
go-version: '1.22'
check-latest: true
if: env.GIT_DIFF
- name: Run benchmark tests
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/auto-format.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ name: Auto Format
on:
pull_request:

permissions: read-all

jobs:
format-go-code:
runs-on: ubuntu-latest
Expand All @@ -13,7 +15,7 @@ jobs:
steps:
- uses: actions/setup-go@v5
with:
go-version: "1.21"
go-version: "1.22"
check-latest: true
- uses: actions/checkout@v4
with:
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/bsr-push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,14 @@ on:
push:
tags:
- "v*.*.*"
permissions: read-all

jobs:
push:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: bufbuild/buf-setup-action@v1.30.0
- uses: bufbuild/buf-setup-action@v1.31.0
# Push Evmos protos to the Buf Schema Registry
- uses: bufbuild/buf-push-action@v1.2.0
with:
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@ on:
branches:
- main
- release/**

permissions: read-all

jobs:
cleanup-runs:
Expand All @@ -20,7 +22,7 @@ jobs:
- uses: actions/checkout@v4
- uses: actions/setup-go@v5
with:
go-version: '1.21'
go-version: '1.22'
check-latest: true
- uses: technote-space/get-diff-action@v6.1.2
id: git_diff
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/changelog.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ on:
branches:
- main
- release/**
permissions: read-all

jobs:
check_diff:
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/check-licenses.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@ name: Check Licenses
on:
pull_request

permissions: read-all

jobs:
check-licenses:
runs-on: ubuntu-latest
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/codeql-analysis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ name: "CodeQL"

on:
pull_request:
permissions: read-all

jobs:
analyze:
Expand Down
6 changes: 4 additions & 2 deletions .github/workflows/consensuswarn.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
name: "Consensus Warn"

permissions: read-all

on:
pull_request_target:
types:
Expand All @@ -10,10 +12,10 @@ on:
jobs:
main:
permissions:
pull-requests: write # For reading the PR and posting comment
pull-requests: write # For reading the PR and posting comment
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: orijtech/consensuswarn@main
with:
roots: 'github.com/evmos/evmos/v16/app.Evmos.DeliverTx,github.com/evmos/evmos/v16/app.Evmos.BeginBlocker,github.com/evmos/evmos/v16/app.Evmos.EndBlocker'
roots: "github.com/evmos/evmos/v18/app.Evmos.DeliverTx,github.com/evmos/evmos/v18/app.Evmos.BeginBlocker,github.com/evmos/evmos/v18/app.Evmos.EndBlocker"
5 changes: 2 additions & 3 deletions .github/workflows/dependencies.yml
Original file line number Diff line number Diff line change
@@ -1,16 +1,15 @@
name: "Dependency Review"
on: pull_request

permissions:
contents: read
permissions: read-all

jobs:
dependency-review:
runs-on: ubuntu-latest
steps:
- uses: actions/setup-go@v5
with:
go-version: '1.21'
go-version: '1.22'
check-latest: true
- name: "Checkout Repository"
uses: actions/checkout@v4
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/docker-push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ on:
push:
tags:
- "v*.*.*"
permissions: read-all

jobs:
docker:
Expand Down
11 changes: 7 additions & 4 deletions .github/workflows/e2e-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,14 +7,15 @@ on:
push:
branches:
- release/**
permissions: read-all

jobs:
test-e2e:
runs-on: ubuntu-latest
steps:
- uses: actions/setup-go@v5
with:
go-version: '1.21'
go-version: '1.22'
check-latest: true
- uses: actions/checkout@v4
- uses: technote-space/get-diff-action@v6.1.2
Expand Down Expand Up @@ -50,12 +51,12 @@ jobs:
"test_grpc_only.py"
"test_ibc.py"
"test_no_abci_resp.py"
"test_osmosis_outpost.py"
"test_str_v2.py"
"test_str_v2_token_factory.py"
"test_precompiles.py"
"test_priority.py"
"test_pruned_node.py"
"test_rollback.py"
"test_stride_outpost.py"
"test_storage_proof.py"
"test_zero_fee.py"
)
Expand All @@ -80,6 +81,8 @@ jobs:
echo "Test matrix: ${TEST_FILES}"
test-nix:
needs: test-files-matrix
permissions:
contents: write
runs-on: ubuntu-latest
strategy:
matrix:
Expand Down Expand Up @@ -125,7 +128,7 @@ jobs:
run: make run-nix-tests
if: env.GIT_DIFF
# Commit gomod2nix changes files back to the repository if necessary
- uses: stefanzweifel/git-auto-commit-action@v5
- uses: stefanzweifel/git-auto-commit-action@v5
with:
commit_message: update gomod2nix.toml file
file_pattern: '*.toml'
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@ on:
push:
tags:
- "v*.*.*"
permissions: read-all

jobs:
goreleaser:
runs-on: ubuntu-latest
Expand All @@ -15,7 +17,7 @@ jobs:
- name: Set up Go
uses: actions/setup-go@v5
with:
go-version: '1.21'
go-version: '1.22'
check-latest: true
- name: release dry run
run: make release-dry-run
Expand Down
9 changes: 6 additions & 3 deletions .github/workflows/labeler.yml
Original file line number Diff line number Diff line change
@@ -1,11 +1,14 @@
name: "Pull Request Labeler"
on:
pull_request:
permissions: read-all

jobs:
triage:
runs-on: ubuntu-latest
permissions:
pull-requests: write # For reading the PR and adding the label
steps:
- uses: actions/labeler@v5
with:
repo-token: "${{ secrets.GITHUB_TOKEN }}"
- uses: actions/labeler@v5
with:
repo-token: "${{ secrets.GITHUB_TOKEN }}"
7 changes: 5 additions & 2 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@ name: Lint
# running if no *.{go, mod, sum} files have been changed.
on:
pull_request:

permissions: read-all

jobs:
golangci:
name: Run golangci-lint
Expand All @@ -13,7 +16,7 @@ jobs:
# Required: setup-go, for all versions v3.0.0+ of golangci-lint
- uses: actions/setup-go@v5
with:
go-version: '1.21'
go-version: '1.22'
check-latest: true
- uses: actions/checkout@v4
- uses: technote-space/get-diff-action@v6.1.2
Expand All @@ -23,7 +26,7 @@ jobs:
go.mod
go.sum
*.toml
- uses: golangci/golangci-lint-action@v4.0.0
- uses: golangci/golangci-lint-action@v5.0.0
with:
# Required: the version of golangci-lint is required and must be specified without patch version: we always use the latest patch version.
version: latest
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/markdown-links.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ on:
branches:
- main
- release/**
permissions: read-all

jobs:
markdown-link-check:
Expand Down
8 changes: 6 additions & 2 deletions .github/workflows/proto.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,13 @@ on:
paths:
- "proto/**"

permissions: read-all

jobs:
build:
runs-on: ubuntu-latest
permissions:
contents: write
steps:
- uses: actions/checkout@v4
- uses: technote-space/get-diff-action@v6.1.2
Expand All @@ -28,7 +32,7 @@ jobs:
timeout-minutes: 5
steps:
- uses: actions/checkout@v4
- uses: bufbuild/buf-setup-action@v1.30.0
- uses: bufbuild/buf-setup-action@v1.31.0
- uses: bufbuild/buf-lint-action@v1
with:
input: "proto"
Expand All @@ -37,7 +41,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: bufbuild/buf-setup-action@v1.30.0
- uses: bufbuild/buf-setup-action@v1.31.0
- uses: bufbuild/buf-breaking-action@v1
with:
input: "proto"
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/security.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
name: Run Gosec
on:
pull_request:
permissions: read-all

jobs:
Gosec:
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/semgrep.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ on:
# Scan changed files in PRs, block on new issues only (existing issues ignored)
pull_request:

permissions: read-all

jobs:
# Update from: https://semgrep.dev/docs/semgrep-ci/sample-ci-configs/#github-actions [removing GH Security Dashboard]
semgrep:
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/slither.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ name: Slither Analysis

on:
pull_request:
permissions: read-all

jobs:
analyze:
Expand All @@ -23,7 +24,7 @@ jobs:
cd contracts && npm i
cp -r node_modules/@openzeppelin .
- name: Run Slither Action
uses: crytic/slither-action@v0.3.1
uses: crytic/slither-action@v0.3.2
continue-on-error: true
id: slither
with:
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/solhint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,12 @@ name: Solhint
on:
pull_request:

permissions: read-all

jobs:
solhint:
name: runner / solhint
runs-on: ubuntu-latest
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: technote-space/get-diff-action@v6.1.2
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/solidity-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,15 @@ on:
branches:
- main
- release/**
permissions: read-all

jobs:
test-solidity:
runs-on: ubuntu-latest
steps:
- uses: actions/setup-go@v5
with:
go-version: '1.21'
go-version: '1.22'
check-latest: true
- uses: actions/checkout@v4
- uses: technote-space/get-diff-action@v6.1.2
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/stale.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,12 @@ name: "Close stale issues & pull requests"
on:
schedule:
- cron: "0 0 * * *"
permissions: read-all

jobs:
stale:
permissions:
pull-requests: write # For reading the PR and adding the label
runs-on: ubuntu-latest
steps:
- uses: actions/stale@v9
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/super-linter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ name: Lint Code Base

on:
pull_request:
permissions: read-all
jobs:
run-lint:
runs-on: ubuntu-latest
Expand All @@ -20,7 +21,7 @@ jobs:
fetch-depth: 0

- name: Lint Code Base
uses: github/super-linter@v5
uses: github/super-linter@v6
env:
LINTER_RULES_PATH: /
YAML_CONFIG_FILE: .yamllint
Expand Down