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

Adds a codec and makes chain reader use it. #11303

Merged
merged 1 commit into from
Dec 1, 2023

Conversation

nolag
Copy link
Contributor

@nolag nolag commented Nov 15, 2023

Adds a codec and makes chain reader use it.

Copy link
Contributor

I see that you haven't updated any CHANGELOG files. Would it make sense to do so?

@nolag nolag force-pushed the rtinianov_encode_decode branch from 0377150 to 2f4c4d9 Compare November 15, 2023 16:55
@nolag nolag force-pushed the rtinianov_encode_decode branch from 2f4c4d9 to 002dedf Compare November 15, 2023 17:02
@nolag nolag force-pushed the rtinianov_encode_decode branch from 002dedf to 3b9d0c1 Compare November 15, 2023 18:36
@nolag nolag force-pushed the rtinianov_encode_decode branch from 3b9d0c1 to e4c8cdc Compare November 27, 2023 20:50
@nolag nolag force-pushed the rtinianov_encode_decode branch from e4c8cdc to 3b1c28c Compare November 28, 2023 16:04
@nolag nolag force-pushed the rtinianov_sizes branch 2 times, most recently from 327534f to 1690e14 Compare November 28, 2023 19:29
Base automatically changed from rtinianov_sizes to BCF-2612-ChainReader-Next November 28, 2023 20:47
@nolag nolag force-pushed the rtinianov_encode_decode branch from 3b1c28c to c89d452 Compare November 29, 2023 20:50
@nolag nolag changed the title Add encoder and decoder for EVM Adds a codec and makes chain reader use it. Nov 29, 2023
@nolag nolag force-pushed the rtinianov_encode_decode branch 6 times, most recently from 5373a97 to 9f0a79c Compare November 30, 2023 20:33
@nolag nolag force-pushed the rtinianov_encode_decode branch from 9f0a79c to 818cd08 Compare November 30, 2023 21:11
@@ -0,0 +1,60 @@
// SPDX-License-Identifier: MIT
pragma solidity ^0.8;
Copy link
Contributor

Choose a reason for hiding this comment

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

+1 for testing this with an actual test smart contract! This is great :-)

@nolag nolag force-pushed the rtinianov_encode_decode branch from 818cd08 to 103a8c5 Compare December 1, 2023 15:15
@cl-sonarqube-production
Copy link

SonarQube Quality Gate

Quality Gate failed

Failed condition 58.6% 58.6% Coverage on New Code (is less than 75%)

See analysis details on SonarQube

@nolag nolag merged commit 3773479 into BCF-2612-ChainReader-Next Dec 1, 2023
84 of 85 checks passed
@nolag nolag deleted the rtinianov_encode_decode branch December 1, 2023 19:39
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