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

chore: adding semgrep #2

Closed
wants to merge 1,099 commits into from
Closed

chore: adding semgrep #2

wants to merge 1,099 commits into from

Conversation

mr-uniswap
Copy link
Contributor

Adding a semgrep configuration

marktoda and others added 30 commits February 9, 2023 11:10
Move jest and typechain to devdeps
* fix: signature sanitization

This commit switches to only sanitizing signatures of normal ECDSA
length (64 or 65 bytes). Other signatures may be eip1271 related so we
leave them be

* fix: comments
This makes it easier to rebuild dutch orders from api responses
feat: Check endTime, startTime, and deadline invariants on build step
This commit adds a helper for calculating dutch decay at a given point
in time
feat: Move deadline check to build step
This commit updates constants with the latest
gouda contracts
* feat: add BSC SupportedChainId

* run prettier
tinaszheng and others added 20 commits October 13, 2023 17:37
fix: use zero price impact instead of throwing error on 100% buy-tax
* Wrapped support on more chains

* Fix code style issues with Prettier

* correction

---------

Co-authored-by: Lint Action <[email protected]>
* v3.2.2

* fix fot token tax retrieval source

* Fix code style issues with Prettier

* revert version change here

---------

Co-authored-by: Lint Action <[email protected]>
This commit updates router addresses the one thats currently used live
on mainnet and goerli
…e16c5'

git-subtree-dir: sdks/sdk-core
git-subtree-mainline: 6f23aee
git-subtree-split: 06e102f
…f71'

git-subtree-dir: sdks/v2-sdk
git-subtree-mainline: f98bfb6
git-subtree-split: 8194848
…032'

git-subtree-dir: sdks/v3-sdk
git-subtree-mainline: 21f3c12
git-subtree-split: 81d6609
…d8daba3aac859865a'

git-subtree-dir: sdks/universal-router-sdk
git-subtree-mainline: e48d790
git-subtree-split: cb9c0f0
…3f187d7'

git-subtree-dir: sdks/router-sdk
git-subtree-mainline: 4792c02
git-subtree-split: bf44f55
…5a3f07cb1'

git-subtree-dir: sdks/uniswapx-sdk
git-subtree-mainline: 77eef6a
git-subtree-split: 7b4375b
…d55dcc14'

git-subtree-dir: sdks/permit2-sdk
git-subtree-mainline: 2ba7498
git-subtree-split: cb84da1
@mr-uniswap mr-uniswap requested a review from adjkant January 10, 2024 16:47
@mr-uniswap mr-uniswap changed the title adding semgrep chore: adding semgrep Jan 10, 2024
@mr-uniswap
Copy link
Contributor Author

@adjkant I'll leave this open until you're ready to merge

@adjkant
Copy link
Collaborator

adjkant commented Mar 13, 2024

Added this change into #1 for simplictly, closing!

@adjkant adjkant closed this Mar 13, 2024
adjkant added a commit that referenced this pull request Mar 13, 2024
adjkant added a commit that referenced this pull request Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.