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.

Sign up for GitHub

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

Jump to bottom

refactor(protocol): remove unused code post Ontake fork #18150

Merged
merged 67 commits into from
Nov 3, 2024
Merged

refactor(protocol): remove unused code post Ontake fork #18150

merged 67 commits into from
Nov 3, 2024

Conversation

dantaik
Copy link
Contributor

@dantaik dantaik commented Sep 20, 2024 •
edited
Loading

There shouldn't be any breaking changes in this PR.

@dantaik @YoGhurt111
@davidtaikocha @bearni95 @taiko-kitty @RogerLamTd @mask-pp @smtmfft @lagunovsky @cyberhorsey @kimo-ice
refactor(protocol): remove unused code post Ontake fork (#18072)
ce76e66 
Co-authored-by: dantaik <dantaik@users.noreply.github.com>
Co-authored-by: gavin <gavin@taiko.xyz>
Co-authored-by: David <david@taiko.xyz>
Co-authored-by: Bernat Canal Garceran <bernatcanal@gmail.com>
Co-authored-by: taiko-bot <160625009+taiko-kitty@users.noreply.github.com>
Co-authored-by: Roger <50648015+RogerLamTd@users.noreply.github.com>
Co-authored-by: maskpp <maskpp266@gmail.com>
Co-authored-by: smtmfft <99081233+smtmfft@users.noreply.github.com>
Co-authored-by: Ivan Lagunovsky <ivan.lagunovsky@gmail.com>
Co-authored-by: jeff <113397187+cyberhorsey@users.noreply.github.com>
Co-authored-by: Karim <karim@taiko.xyz>
Co-authored-by: Karim <162329697+kimo-ice@users.noreply.github.com>
@dantaik dantaik added area.protocol option.pinned Will not be marked as stale automatically option.do-not-merge labels Sep 20, 2024
@openzeppelin-code OpenZeppelin Code
Copy link

openzeppelin-code bot commented Sep 20, 2024 •
edited
Loading

refactor(protocol): remove unused code post Ontake fork

Generated at commit: a76f2ccab6c53a0313c44e5ba529029027e12cad

🚨 Report Summary

Severity Level Results
Contracts Critical
High
Medium
Low
Note
Total		 3
3
0
7
42
55
Dependencies Critical
High
Medium
Low
Note
Total		 0
0
0
0
0
0

For more details view the full report in OpenZeppelin Code Inspector

dantaik and others added 22 commits September 22, 2024 11:04
@dantaik
Merge branch 'main' into ontake_cleanup2
1f4b014
@dantaik
Merge branch 'main' into ontake_cleanup2
671dd42
@dantaik
Merge branch 'main' into ontake_cleanup2
1cf87c9
@dantaik
Merge branch 'main' into ontake_cleanup2
4b46a8c
@dantaik
Merge branch 'main' into ontake_cleanup2
86852f0
@dantaik
Merge branch 'main' into ontake_cleanup2
5e50481
@dantaik
fix(protocol): remove "payable" from ProverSet.sol (#18209)
44e5184
@dantaik
fix comments
1dcc18a
@dantaik
Update ComposeVerifier.sol
db376c8
@dantaik
Merge branch 'main' into ontake_cleanup2
93241d5
@dantaik
Merge branch 'main' into ontake_cleanup2
8e076c1
@dantaik
refactor(protocol): extract SgxVerifierBase to be reused (#18231)
c9982e6
@dantaik
Merge branch 'main' into ontake_cleanup2
a77d40a
@dantaik
feat(protocol): migrate Nethermind's preconf contracts here (#18234)
7fbfe84 
Co-authored-by: dantaik <dantaik@users.noreply.github.com>
@davidtaikocha @dantaik
docs(protocol): update Hekla deployments (#18257)
035acb2
@YoGhurt111 @dantaik
docs(taiko-client): update readme how to do integration test (#18256)
e8b1454
@davidtaikocha @dantaik
docs(protocol): update mainnet deployment (#18258)
6f668e5
@RogerLamTd @dantaik
chore(docs-site): remove banner (#18267)
f91c2b5
@taiko-kitty @dantaik
chore(main): release docs-site 1.12.3 (#18268)
129960a
@dantaik
chore(protocol): rename B_PRECONF_REGISTRY to B_PRECONF_TASK_MANAGER (#…
438d139 
…18271)
@dantaik
Merge branch 'main' into ontake_cleanup2
4c9df8c
@dantaik
fix(protocol): update slotB.lastUnpausedAt by both general unpause …
90f014b 
…and proving unpause (#18276)

Co-authored-by: dantaik <dantaik@users.noreply.github.com>
@dantaik dantaik marked this pull request as ready for review October 23, 2024 03:33
@dantaik dantaik disabled auto-merge November 2, 2024 03:43
@dantaik dantaik added this pull request to the merge queue Nov 3, 2024
Merged via the queue into main with commit 8543cec Nov 3, 2024
5 checks passed
@dantaik dantaik deleted the ontake_cleanup2 branch November 3, 2024 05:46
@taiko-kitty taiko-kitty mentioned this pull request Nov 3, 2024
Closed
davidtaikocha added a commit that referenced this pull request Nov 3, 2024
@davidtaikocha
feat(taiko-client): client changes based on #18150
de894ac
github-merge-queue bot pushed a commit that referenced this pull request Nov 3, 2024
@davidtaikocha
feat(taiko-client): client changes based on #18150 (#18350)
ddc6473
This was referenced Nov 3, 2024
Closed
Closed
Closed
Closed
Closed
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidtaikocha davidtaikocha davidtaikocha approved these changes

@YoGhurt111 YoGhurt111 YoGhurt111 approved these changes

@adaki2004 adaki2004 Awaiting requested review from adaki2004

@smtmfft smtmfft Awaiting requested review from smtmfft

Assignees
No one assigned
Labels
area.protocol
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@dantaik @YoGhurt111 @davidtaikocha @RogerLamTd @taiko-kitty