-
Notifications
You must be signed in to change notification settings - Fork 4
Fit to template repo #48
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
Merged
Merged
Changes from all commits
Commits
Show all changes
75 commits
Select commit
Hold shift + click to select a range
f9101f5
first commit
godzillaba 5082cc5
update readme
godzillaba cde0e5f
package.json
godzillaba 2f4dcd9
hardhat init
godzillaba b0e6f9b
foundry init
godzillaba 6d422a4
Counter
godzillaba bc5f3b8
package.json
godzillaba 239a06a
Merge branch 'main' into hardhat-foundry-ethers-v6
godzillaba aa31614
Merge pull request #1 from OffchainLabs/hardhat-foundry-ethers-v6
godzillaba 87ec434
sync compiler settings
godzillaba c28bb95
typo
godzillaba 20d50a0
update readme
godzillaba bf93fee
Formatting (#3)
godzillaba 0c8c0f8
Misc CI (#5)
godzillaba a79e748
Fork tests (#7)
godzillaba c6c05c9
e2e test (#8)
godzillaba c12d75f
Audit (#9)
godzillaba 8aa9ed6
minimal-publish (#10)
godzillaba b199f71
Update README.md
godzillaba 9b4f94a
publish.bash
godzillaba 0772589
separate files for jobs (#12)
godzillaba 245bb44
fix: yarn and misc improvements (#13)
gzeoneth 2770d48
document disabling e2e (#14)
godzillaba 380fc79
fix: script path
gzeoneth 160fe16
Merge pull request #17 from OffchainLabs/gzeoneth-patch-1
gzeoneth f741bd2
minimal-install
godzillaba 029c435
Merge pull request #18 from OffchainLabs/minimal-install
gzeoneth f57a1e0
Lighter e2e (#19)
godzillaba 577f8b6
add gas snapshot to fix
godzillaba b1c75fe
fix print contracts script (#20)
godzillaba 910ac04
error if no contracts found (#21)
godzillaba 92f6f30
unused import
godzillaba e5f4559
improve inspect (#22)
godzillaba 64a0dfa
Delete sample files script (#23)
godzillaba f2bddcb
fix bash scripts (#24)
godzillaba dc0ac29
Unused errors checker (#26)
gvladika d6a159f
use hardhat-sizer (#29)
godzillaba 8f9ee56
move template scripts to their own dir (#28)
godzillaba 3d6c1cb
simplify test scripts (#25)
godzillaba fe7daba
fix audit:ci
godzillaba 84db337
Merge pull request #31 from OffchainLabs/audit-ci-fix
gzeoneth 6cff375
Mocks + DoubleProvider (#30)
godzillaba 3b432ae
explicit formatter config (#34)
godzillaba b3edd41
uncomment e2e ci (#36)
godzillaba d553ef0
use stable foundry (#37)
godzillaba d7b043b
use node v22, run `yarn upgrade`, fix tsconfig (#38)
godzillaba dac51dc
Bump arbitrum sdk (#39)
godzillaba 7ea120b
fix: audit (#40)
godzillaba 7262aa5
import template
godzillaba 3993013
install contract deps through npm and fix contract import paths
godzillaba 8f354a2
fmt
godzillaba cf3ac1d
fix FundRouters.test.ts
godzillaba 265a10f
fmt
godzillaba ca58a10
install viem
godzillaba d0295b3
fix ChildToParentMessageRedeemer.ts
godzillaba fc33935
lib.ts
godzillaba 6016f18
checkAndRouteFunds.ts
godzillaba 2c2aed5
bump ts, couple files
godzillaba 90cb2d6
add yargs, cli
godzillaba 129f3f7
fix e2e tests
godzillaba 9fe4fb4
no lint on build
godzillaba 679e105
fix print-contracts script
godzillaba e06b9fe
format
godzillaba 7748674
fix
godzillaba 663d35f
fix lint
godzillaba 003e982
include total supply in deployment of test token
godzillaba eb7e303
remove test token minting
godzillaba 1c6473f
try fix tests
godzillaba fca328f
fix tests
godzillaba 7d59c9f
use ha/traffic-flag branch of testnode
godzillaba 25c01a3
Revert "use ha/traffic-flag branch of testnode"
godzillaba 90c4dc8
audit fix
godzillaba 32221a5
remove publish helpers
godzillaba db9abdc
Merge remote-tracking branch 'template/ha/remove-publish' into local/…
godzillaba c7c27d7
remove publish
godzillaba File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
lib/ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,91 @@ | ||
module.exports = { | ||
env: { | ||
commonjs: true, | ||
es6: true, | ||
node: true, | ||
}, | ||
plugins: ['prettier'], | ||
extends: [ | ||
'eslint:recommended', | ||
'plugin:prettier/recommended', // Enables eslint-plugin-prettier and displays prettier errors as ESLint errors. Make sure this is always the last configuration in the extends array. | ||
], | ||
parserOptions: { | ||
ecmaVersion: 2018, // Allows for the parsing of modern ECMAScript features | ||
sourceType: 'module', // Allows for the use of imports | ||
}, | ||
rules: { | ||
'prettier/prettier': 'error', | ||
'no-unused-vars': 'off', | ||
'prefer-const': [2, { destructuring: 'all' }], | ||
'object-curly-spacing': ['error', 'always'], | ||
}, | ||
overrides: [ | ||
{ | ||
// this config is run against test files (same as the one bellow but not limited to `src` folder) | ||
files: ['*.ts', '*.tsx'], | ||
parser: '@typescript-eslint/parser', | ||
extends: [ | ||
'eslint:recommended', | ||
'plugin:@typescript-eslint/recommended', | ||
'plugin:prettier/recommended', | ||
], | ||
plugins: ['@typescript-eslint', 'prettier'], | ||
rules: { | ||
'no-empty-pattern': 'warn', | ||
'prettier/prettier': ['error', { singleQuote: true }], | ||
'@typescript-eslint/member-delimiter-style': ['off'], | ||
'@typescript-eslint/no-explicit-any': ['off'], | ||
'@typescript-eslint/no-use-before-define': ['off'], | ||
'@typescript-eslint/no-non-null-assertion': ['off'], | ||
'@typescript-eslint/ban-ts-comment': ['warn'], | ||
'@typescript-eslint/no-unused-vars': [ | ||
'warn', | ||
{ | ||
argsIgnorePattern: '^_', | ||
varsIgnorePattern: '^_', | ||
caughtErrorsIgnorePattern: '^_', | ||
}, | ||
], | ||
'no-implicit-coercion': 'error', | ||
}, | ||
}, | ||
{ | ||
files: ['src/**/*.ts', 'src/**/*.tsx'], | ||
parser: '@typescript-eslint/parser', | ||
parserOptions: { | ||
project: 'tsconfig.json', | ||
}, | ||
extends: [ | ||
'eslint:recommended', | ||
'plugin:@typescript-eslint/recommended', | ||
'plugin:prettier/recommended', | ||
], | ||
plugins: ['@typescript-eslint', 'prettier', '@typescript-eslint/tslint'], | ||
rules: { | ||
'no-empty-pattern': 'warn', | ||
'prettier/prettier': ['error', { singleQuote: true }], | ||
'@typescript-eslint/member-delimiter-style': ['off'], | ||
'@typescript-eslint/no-explicit-any': ['off'], | ||
'@typescript-eslint/no-use-before-define': ['off'], | ||
'@typescript-eslint/no-non-null-assertion': ['off'], | ||
'@typescript-eslint/ban-ts-comment': ['warn'], | ||
'@typescript-eslint/no-unused-vars': [ | ||
'warn', | ||
{ | ||
argsIgnorePattern: '^_', | ||
varsIgnorePattern: '^_', | ||
caughtErrorsIgnorePattern: '^_', | ||
}, | ||
], | ||
'@typescript-eslint/tslint/config': [ | ||
'error', | ||
{ | ||
rules: { 'strict-comparisons': true }, | ||
}, | ||
], | ||
'no-implicit-coercion': 'error', | ||
'@typescript-eslint/no-shadow': ['error'], | ||
}, | ||
}, | ||
], | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,18 +1,36 @@ | ||
RewardDistributorTest:testBlockGasLimit() (gas: 22830786) | ||
RewardDistributorTest:testConstructor() (gas: 533016) | ||
RewardDistributorTest:testConstructorDoesNotAcceptEmpty() (gas: 69042) | ||
RewardDistributorTest:testConstructorDoesNotAcceptPastLimit() (gas: 1636148) | ||
RewardDistributorTest:testDistributeAndUpdateRecipients() (gas: 4276467) | ||
RewardDistributorTest:testDistributeAndUpdateRecipientsBadPrevious() (gas: 2096822) | ||
RewardDistributorTest:testDistributeAndUpdateRecipientsNotOwner() (gas: 2099732) | ||
RewardDistributorTest:testDistributeRewards() (gas: 653269) | ||
RewardDistributorTest:testDistributeRewardsDoesNotDistributeToEmpty() (gas: 532370) | ||
RewardDistributorTest:testDistributeRewardsDoesNotDistributeToWrongCount() (gas: 537165) | ||
RewardDistributorTest:testDistributeRewardsDoesNotDistributeWrongRecipients() (gas: 535861) | ||
RewardDistributorTest:testDistributeRewardsDoesRefundsOwner() (gas: 937648) | ||
RewardDistributorTest:testDistributeRewardsFailsForEmptyRD() (gas: 533039) | ||
RewardDistributorTest:testDistributeRewardsFailsToRefundsOwner() (gas: 662631) | ||
RewardDistributorTest:testHashAddresses() (gas: 87649) | ||
RewardDistributorTest:testLowSend() (gas: 658025) | ||
RewardDistributorTest:testUncheckedInc() (gas: 651) | ||
RewardDistributorTest:testUpdateDoesNotAcceptInvalidValues() (gas: 1018294) | ||
ArbChildToParentRewardRouterTest:testSanityCheck() (gas: 257342) | ||
ArbChildToParentRewardRouterTest:testSendNative(uint64) (runs: 256, μ: 44649, ~: 44780) | ||
ArbChildToParentRewardRouterTest:testSendToken(uint256) (runs: 256, μ: 99537, ~: 99238) | ||
ChildToParentRewardRouterTest:testCantRouteTokenTooSoon() (gas: 107176) | ||
ChildToParentRewardRouterTest:testCantRouteTokensWhenSetToNativeOnly() (gas: 8624) | ||
ChildToParentRewardRouterTest:testCantrouteNativeFundsTooSoon() (gas: 100623) | ||
ChildToParentRewardRouterTest:testDistributionUpdate() (gas: 206068) | ||
ChildToParentRewardRouterTest:testRouteNativeFunds() (gas: 78065) | ||
ChildToParentRewardRouterTest:testRouteToken() (gas: 79806) | ||
ChildToParentRewardRouterTest:testZeroAddressConstructorCheck() (gas: 111253) | ||
OpChildToParentRewardRouterTest:testSanityCheck() (gas: 46295) | ||
OpChildToParentRewardRouterTest:testSendNative(uint64) (runs: 256, μ: 23548, ~: 23679) | ||
OpChildToParentRewardRouterTest:testSendToken(uint256) (runs: 256, μ: 93099, ~: 92788) | ||
ParentToChildRewardRouterTest:testCantRouteTooSoon() (gas: 93093) | ||
ParentToChildRewardRouterTest:testRevertsWithInsufficientValue() (gas: 37185) | ||
ParentToChildRewardRouterTest:testrouteNativeFunds() (gas: 89285) | ||
RewardDistributorTest:testBlockGasLimit() (gas: 69524218) | ||
RewardDistributorTest:testConstructor() (gas: 1023688) | ||
RewardDistributorTest:testConstructorDoesNotAcceptEmpty() (gas: 72612) | ||
RewardDistributorTest:testConstructorDoesNotAcceptPastLimit() (gas: 3124911) | ||
RewardDistributorTest:testConstructorInputLengthMismatch() (gas: 252338) | ||
RewardDistributorTest:testDistributeAndUpdateRecipients() (gas: 6231656) | ||
RewardDistributorTest:testDistributeAndUpdateRecipientsBadPrevious() (gas: 4007775) | ||
RewardDistributorTest:testDistributeAndUpdateRecipientsNotOwner() (gas: 4018821) | ||
RewardDistributorTest:testDistributeRewards(uint256) (runs: 256, μ: 1130540, ~: 1130540) | ||
RewardDistributorTest:testDistributeRewardsDoesNotDistributeToEmpty() (gas: 1011249) | ||
RewardDistributorTest:testDistributeRewardsDoesNotDistributeToWrongCount() (gas: 1017358) | ||
RewardDistributorTest:testDistributeRewardsDoesNotDistributeWrongRecipients() (gas: 1015863) | ||
RewardDistributorTest:testDistributeRewardsDoesNotDistributeWrongWeights() (gas: 1015179) | ||
RewardDistributorTest:testDistributeRewardsDoesRefundsOwner(uint256) (runs: 256, μ: 2133067, ~: 2133067) | ||
RewardDistributorTest:testDistributeRewardsFailsToRefundsOwner() (gas: 1144021) | ||
RewardDistributorTest:testDistributeRewardsInputLengthMismatch() (gas: 1012801) | ||
RewardDistributorTest:testHashAddresses() (gas: 84052) | ||
RewardDistributorTest:testLowSend(uint256) (runs: 256, μ: 1252855, ~: 1252855) | ||
RewardDistributorTest:testUncheckedInc() (gas: 5127) | ||
RewardDistributorTest:testUpdateDoesNotAcceptInvalidValues() (gas: 2110869) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
name: Audit | ||
|
||
on: | ||
pull_request: | ||
workflow_dispatch: | ||
|
||
jobs: | ||
audit: | ||
name: Audit | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Checkout | ||
uses: actions/checkout@v4 | ||
|
||
- name: Set up Node.js | ||
uses: actions/setup-node@v4 | ||
with: | ||
node-version: 22 | ||
|
||
- name: Install Foundry | ||
uses: foundry-rs/foundry-toolchain@v1 | ||
with: | ||
version: stable | ||
|
||
- run: yarn | ||
- run: yarn audit:ci |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
name: Lint | ||
|
||
on: | ||
pull_request: | ||
workflow_dispatch: | ||
|
||
jobs: | ||
lint: | ||
name: Lint | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Checkout | ||
uses: actions/checkout@v4 | ||
|
||
- name: Set up Node.js | ||
uses: actions/setup-node@v4 | ||
with: | ||
node-version: 22 | ||
|
||
- name: Install Foundry | ||
uses: foundry-rs/foundry-toolchain@v1 | ||
with: | ||
version: stable | ||
|
||
- run: yarn && yarn build | ||
- run: yarn lint |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
name: Test Fork | ||
|
||
on: | ||
pull_request: | ||
workflow_dispatch: | ||
|
||
jobs: | ||
test-fork: | ||
name: Test Fork | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Checkout | ||
uses: actions/checkout@v4 | ||
|
||
- name: Set up Node.js | ||
uses: actions/setup-node@v4 | ||
with: | ||
node-version: 22 | ||
|
||
- name: Install Foundry | ||
uses: foundry-rs/foundry-toolchain@v1 | ||
with: | ||
version: stable | ||
|
||
- run: yarn | ||
- run: ETH_FORK_URL=${{ secrets.ETH_FORK_URL }} ARB_FORK_URL=${{ secrets.ARB_FORK_URL }} yarn test:fork |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we wanna keep these commented out workflows?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah i think we should just to keep the diff with the template smaller