Skip to content

ci: add Slither static-analysis #103

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 16 commits into from
May 8, 2025
Merged

ci: add Slither static-analysis #103

merged 16 commits into from
May 8, 2025

Conversation

mssassii
Copy link
Contributor

No description provided.

@mssassii mssassii requested review from zimpha and Thegaram April 30, 2025 08:45
@Thegaram Thegaram changed the title CI: add Solhint + Prettier linting and Slither static-analysis ci: add Slither static-analysis May 6, 2025
@mssassii mssassii merged commit cc6fc1b into main May 8, 2025
5 checks passed
@mssassii mssassii deleted the ci/add-lint-and-slither branch May 8, 2025 09:15
Thegaram added a commit that referenced this pull request Jun 26, 2025
Co-authored-by: Péter Garamvölgyi <[email protected]>
alexanvl pushed a commit to DogeOS69/scroll-contracts that referenced this pull request Jul 31, 2025
* hotfix: forbid enforced gateway (scroll-tech#101)

Co-authored-by: Péter Garamvölgyi <[email protected]>

* chore: bump version 2.0.1 (scroll-tech#102)

* ci: bump GitHub Action versions (scroll-tech#107)

* ci: add Slither static-analysis  (scroll-tech#103)

Co-authored-by: Péter Garamvölgyi <[email protected]>

* new `ScrollChainCommitmentVerifier` (scroll-tech#95)

Co-authored-by: Péter Garamvölgyi <[email protected]>

* build(deps): bump base-x from 3.0.9 to 3.0.11 (scroll-tech#108)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* add L2SystemContract (scroll-tech#114)

* feat: add e2e devnet deployment scripts (scroll-tech#97)

* feat: add pause controller (scroll-tech#117)

* feat: consider EIP-7623 floor gas in message queue (scroll-tech#118)

* feat: add L1SystemConfig events (scroll-tech#119)

* feat: require top-level call to commitAndFinalizeBatch (scroll-tech#120)

* feat: add new zkevm verifier for Feynman upgrade (scroll-tech#123)

* feat(feynman): update L1GasPriceOracle (scroll-tech#124)

* remove unused contracts and functions (scroll-tech#110)

Co-authored-by: Péter Garamvölgyi <[email protected]>

* fix: allow only top level call in commitAndFinalizeBatch (scroll-tech#126)

* feat: support resetPauseCooldownPeriod (scroll-tech#127)

* fix conflicts part 1

fix conflicts part2

fix conflict part3

fix compile part1

fix compile part2

fix compile part3

register V8 verifier: feynman upgrade

todo revert this after test

fix rollup-config missing endpoint

remove debug log

add -vvvv

set batch size to 10

some fix

fix read private key

use a small batch size 5, so the l2-sequencer won't get stuck

* fix: read from file still required while run gen-configs

* improve verify.sh

* use deterministic foundry version v1.3.0-rc2

* fix rollup-config.json gas_price_oracle_contract_address

* remove --resume

* set BATCH_SIZE=7

* fix bug: missing L2_SYSTEM_CONFIG

other missing fields in genesis.json

* set genesisStateRoot default to null

set missingHeaderFieldsSHA256 default to null //TODO

* replace scrolltech with dogeos69

* verify.sh add L2SystemConfig

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Xi Lin <[email protected]>
Co-authored-by: Péter Garamvölgyi <[email protected]>
Co-authored-by: Souhail M. <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Morty <[email protected]>
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.

3 participants