Skip to content

Only run size check on contract source#5

Open
gretzke wants to merge 22 commits intogretzke:mainfrom
Uniswap:gretzke-patch-1
Open

Only run size check on contract source#5
gretzke wants to merge 22 commits intogretzke:mainfrom
Uniswap:gretzke-patch-1

Conversation

@gretzke
Copy link
Copy Markdown
Owner

@gretzke gretzke commented Feb 4, 2026

No description provided.

gretzke and others added 22 commits July 2, 2024 21:12
* Dev (#2)

* Remove default MIT license

* Remove libraries

Co-authored-by: Madeveda <Madeveda@users.noreply.github.com>

* forge install: forge-gas-snapshot

* add testing info

* remove versioning

* add gasmetering to tests (#1)

* update solidity version

* add issue templates

* add coverage script

* add formatting rules

* add RPCs

---------

Co-authored-by: Madeveda <Madeveda@users.noreply.github.com>

* Ignore coverage report, increase coverage, adjust CI (#4)

* Ignore coverage report, increase coverage, adjust CI

* remove test helper

* Dev (#5)

* Ignore coverage report, increase coverage, adjust CI

* remove test helper

* Add testing as option

* Add requirements for merging

---------

Co-authored-by: Madeveda <Madeveda@users.noreply.github.com>
Signed-off-by: mr-uniswap <144828035+mr-uniswap@users.noreply.github.com>
Signed-off-by: mr-uniswap <144828035+mr-uniswap@users.noreply.github.com>
Signed-off-by: mr-uniswap <144828035+mr-uniswap@users.noreply.github.com>
Signed-off-by: mr-uniswap <144828035+mr-uniswap@users.noreply.github.com>
* Dev (#8)

* Dev (#2)

* Remove default MIT license

* Remove libraries

Co-authored-by: Madeveda <Madeveda@users.noreply.github.com>

* forge install: forge-gas-snapshot

* add testing info

* remove versioning

* add gasmetering to tests (#1)

* update solidity version

* add issue templates

* add coverage script

* add formatting rules

* add RPCs

---------

Co-authored-by: Madeveda <Madeveda@users.noreply.github.com>

* Ignore coverage report, increase coverage, adjust CI

---------

Signed-off-by: Daniel Gretzke <daniel@gretzke.de>
Co-authored-by: Madeveda <Madeveda@users.noreply.github.com>

* formatting

* formatting

* require review from security team on workflow changes

---------

Signed-off-by: Daniel Gretzke <daniel@gretzke.de>
Co-authored-by: Madeveda <Madeveda@users.noreply.github.com>
Signed-off-by: mr-uniswap <144828035+mr-uniswap@users.noreply.github.com>
* fix a few bugs with doc generation

* add additional compiler profiles to template
Signed-off-by: Daniel Gretzke <daniel@gretzke.de>
* use foundry stable version

Signed-off-by: Daniel Gretzke <daniel@gretzke.de>

* use stable version of foundry for pre-commit

* use latest ubuntu version

---------

Signed-off-by: Daniel Gretzke <daniel@gretzke.de>
Co-authored-by: vimageDE <m.wierzimok@vimage.de>
* delete branching strategy

* Replace --isolate with annotation

* IR compilation and tests

* Add link to latest contributing doc

* Add link to foundrybook testing section for best practices

* triple slash natspec

* naming collision avoidance

* Add section on Solidity versioning

* add squash merge requirement

* Update gas snapshot section and reflect changes in template

* Revert "delete branching strategy"

This reverts commit f6ad557.

* Update branching strategy

* Add Dependency Management section

* Add release section

* clarify single line natspec comment

* updated branching strategy

* Add bytecode hash
Signed-off-by: Daniel Gretzke <daniel@gretzke.de>
Updated Forge build command to include specific source files.

Signed-off-by: Daniel Gretzke <daniel@gretzke.de>
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.

5 participants