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

feat: use sqlite on lastgersync #150

Merged
merged 4 commits into from
Nov 5, 2024
Merged

Conversation

arnaubennassar
Copy link
Contributor

No description provided.

Copy link
Collaborator

@Stefan-Ethernal Stefan-Ethernal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally LGTM!

@Stefan-Ethernal Stefan-Ethernal changed the title feat use sqlite on lastgersync feat: use sqlite on lastgersync Nov 4, 2024
Copy link
Contributor

@vcastellm vcastellm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Co-authored-by: Goran Rojovic <100121253+goran-ethernal@users.noreply.github.com>
Copy link

sonarqubecloud bot commented Nov 5, 2024

@arnaubennassar arnaubennassar merged commit 6d8dd74 into develop Nov 5, 2024
10 checks passed
@arnaubennassar arnaubennassar deleted the feat/lastger-sqlite branch November 5, 2024 18:42
ToniRamirezM added a commit that referenced this pull request Nov 6, 2024
* feat: calculate acc input hash locally

* fix: test

* feat: Add `metadata` field on the certificate (#151)

* feat: use metadata field on certificate

* fix: lint and UT

* fix: comments

* fix: test

* fix: use calculated acc input hash in input prover

* fix: use calculated acc input hash in input prover

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* feat: change timestamp

* feat: change timestamp

* minor improvements on the config (#149)

* fix: revert changes on calculation of merkel proof (#156)

* feat: change timestamp

* feat: update zkevm-ethtx-manager to v0.2.1 (#153)

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* feat: refactor

* fix: typo

* fix: lock

* feat: use sqlite on lastgersync (#150)

* feat use sqlite on lastgersync

* apply requests

* rm tree migrations

* Update lastgersync/processor.go

Co-authored-by: Goran Rojovic <100121253+goran-ethernal@users.noreply.github.com>

---------

Co-authored-by: Goran Rojovic <100121253+goran-ethernal@users.noreply.github.com>

* feat: use sqlite on claimsponsor (#157)

* feat use sqlite on claimsponsor

* wip

* pass UTs

* fix identation

* fix identation

* rm cover.out

* rm tree migrations

* make err a var

* chore: bump cdk-erigon to v2.1.2

* Revert "chore: bump cdk-erigon to v2.1.2"

This reverts commit a5422d2.

---------

Co-authored-by: Goran Rojovic <100121253+goran-ethernal@users.noreply.github.com>
Co-authored-by: Arnau Bennassar <arnaubennassar5@gmail.com>
Co-authored-by: Joan Esteban <129153821+joanestebanr@users.noreply.github.com>
Co-authored-by: Victor Castell <0x@vcastellm.xyz>
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