Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
sonarcloud[bot] created a comment on a pull request on 0xPolygon/cdk
## [](https://sonarcloud.io/dashboard?id=...
goran-ethernal created a review comment on a pull request on 0xPolygon/cdk
Fixed: https://github.com/0xPolygon/cdk/pull/22/commits/335b15683579f711848d5c3f51de1ab7f26a3a75
Stefan-Ethernal created a review comment on a pull request on 0xPolygon/cdk
Nitpick, simplification: ```suggestion // Get new last verified batch number from L1 lastVerifiedBatchNumber, err := a.etherman.GetLatestVerifiedBatchNum() ```
Stefan-Ethernal created a review comment on a pull request on 0xPolygon/cdk
Nitpick: extract the `busy` string to a var?
Stefan-Ethernal created a review comment on a pull request on 0xPolygon/cdk
A bit unrelated to this PR, but should we export these fields? Warning analysis is complaining.
Stefan-Ethernal created a review on a pull request on 0xPolygon/cdk
Generally LGTM, leaving a couple of comments to consider.
vcastellm closed a pull request on 0xPolygon/cdk
test: l2 to L1 withdrawal
* L2 to L1 withdrawal test * Refactor tests to reduce complexity * Bump `kurtosis-cdk` version to `v0.2.15` * Improve CI by dumping enclave logs + files in kurtosis
joanestebanr created a review comment on a pull request on 0xPolygon/cdk
it looks like in that case you don't set properly `lastCertificateBlock`
goran-ethernal created a review comment on a pull request on 0xPolygon/cdk
We are talking with the agglayer team, so they will probably add a new rpc point for this.
sonarcloud[bot] created a comment on a pull request on 0xPolygon/cdk
## [](https://sonarcloud.io/dashboard?id=...
vcastellm created a review comment on a pull request on 0xPolygon/cdk
it's not needed but I prefer to leave it for convenience, wdyt?
vcastellm created a review comment on a pull request on 0xPolygon/cdk
introduced in the latest commit