-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature/swaps via call #29
Open
AleksiGer
wants to merge
54
commits into
master
Choose a base branch
from
feature/swaps-via-call
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
34072f9
del DexType enum and add dexRouter
AleksiGer c5d248c
natspec
AleksiGer 9cbdc29
add InsufficientAmount error
AleksiGer 9b3f94f
add additional after swap check
AleksiGer 6e9d021
implement _performSwap fn
AleksiGer e924f8d
delete unused param
AleksiGer 1a3b71f
add receiving tokens by the user
AleksiGer f85c295
delete unused
AleksiGer d38fd96
add new tokens
AleksiGer 3ecfbd7
add args in forks
AleksiGer b6f43f2
add tests for DexSwap
AleksiGer f654b9a
delete useless checks and calls
AleksiGer aa5cbe6
add new tokens
AleksiGer 5d42039
add avalanche fork
AleksiGer f25392e
add polygon & avalanche tests
AleksiGer eb1b154
add forks and funcs for them
AleksiGer 47ab350
optimize comparison
AleksiGer 980bc7e
optimize conditions
AleksiGer 0f85554
delete comments and unused imports
AleksiGer dd5b309
delete abicoder v2 because it already presents since 0.8.0
AleksiGer b150278
Merge branch 'master' into feature/swaps-via-call
lufaque 2757f1a
fix naming issues
lufaque 543ec88
new test suit base
lufaque 11d1b13
uni v3 test
lufaque 36249f5
add view
AleksiGer 3c2077e
update naming
lufaque 4538e9f
update dex swap, add univ3 and sushi swap tests
lufaque 07eeb59
fix typo
AleksiGer 69e58cc
update dex swap, add univ3 and sushi swap tests
lufaque ea28efc
Merge remote-tracking branch 'origin/feature/swaps-via-call' into fea…
lufaque 23870e5
sushi swap, uniswap tests
lufaque 0eeda0f
uni, quickswap and alienbase tests
AleksiGer 489a2ef
add polygon, avalanche, optimism
AleksiGer 4039e8d
Merge branch 'feature/swaps-via-call' of https://github.com/concero/c…
AleksiGer 84e84ba
reorder tests and fix name
AleksiGer 5dff768
remove comments
lufaque a41d35f
fix env names
lufaque 01737df
remove unused imports
lufaque 7298656
reformat label
AleksiGer 5f31617
Merge branch 'feature/swaps-via-call' of https://github.com/concero/c…
AleksiGer 6767f77
setup solhint
lufaque 3a654d4
Merge remote-tracking branch 'origin/feature/swaps-via-call' into fea…
lufaque 6f6634e
update set contracts task
lufaque e194fe9
rename dexData to dexCallData
AleksiGer 178e2bc
Merge branch 'feature/swaps-via-call' of https://github.com/concero/c…
AleksiGer 83ce309
rename dexData to dexCallData
AleksiGer 338da27
add tokens
AleksiGer f929cfb
add wrapped to whitelist
AleksiGer bf18749
add new tests for native and wrapped
AleksiGer 7fc08cb
update deploy dex swap script
lufaque 14a17d8
add pangolin
AleksiGer 598bc34
add tests for pangolin avalanche
AleksiGer 69f323d
rename
AleksiGer 388c972
Merge branch 'feature/swaps-via-call' of https://github.com/concero/c…
AleksiGer 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 contains 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 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 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,51 +1,34 @@ | ||
{ | ||
"extends": "solhint:recommended", | ||
"plugins": [ | ||
"chainlink-solidity" | ||
], | ||
"plugins": ["chainlink-solidity"], | ||
"rules": { | ||
"prettier/prettier": "error", | ||
"no-unused-vars": "error", | ||
"contract-name-camelcase": "error", | ||
"event-name-camelcase": "error", | ||
"func-name-mixedcase": "error", | ||
"func-param-name-mixedcase": "error", | ||
"modifier-name-mixedcase": "error", | ||
"imports-on-top": "error", | ||
"compiler-version": [ | ||
"off", | ||
"^0.8.20" | ||
], | ||
"compiler-version": ["off", "^0.8.0"], | ||
"const-name-snakecase": "off", | ||
"constructor-syntax": "error", | ||
"var-name-mixedcase": "off", | ||
"avoid-low-level-calls": "off", | ||
"chainlink-solidity/no-block-single-if-reverts": "off", | ||
"no-inline-assembly": "off", | ||
"func-visibility": [ | ||
"error", | ||
{ | ||
"ignoreConstructors": true | ||
} | ||
], | ||
"max-line-length": [ | ||
"error", | ||
160 | ||
], | ||
"not-rely-on-time": "off", | ||
"no-empty-blocks": "warn", | ||
"quotes": [ | ||
"error", | ||
"double" | ||
], | ||
"no-empty-blocks": "off", | ||
"reason-string": [ | ||
"warn", | ||
{ | ||
"maxLength": 64 | ||
} | ||
], | ||
"named-return-values": "off", | ||
"chainlink-solidity/prefix-internal-functions-with-underscore": "warn", | ||
"chainlink-solidity/prefix-private-functions-with-underscore": "warn", | ||
"chainlink-solidity/inherited-constructor-args-not-in-contract-definition": "warn", | ||
"chainlink-solidity/no-block-single-if-reverts": "off", | ||
"chainlink-solidity/explicit-returns": "warn" | ||
"chainlink-solidity/prefix-storage-variables-with-s-underscore": "warn", | ||
"chainlink-solidity/prefix-immutable-variables-with-i": "warn", | ||
"chainlink-solidity/all-caps-constant-storage-variables": "warn", | ||
"chainlink-solidity/no-hardhat-imports": "warn", | ||
"chainlink-solidity/inherited-constructor-args-not-in-contract-definition": "warn" | ||
} | ||
} |
This file contains 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 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 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
Oops, something went wrong.
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.
is this sushi dex or sushi token?