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

Execution engine changelog #5046

Merged

Conversation

zajko
Copy link
Contributor

@zajko zajko commented Dec 19, 2024

Backfilling execution_engine backlog. Fixing json serialization issue in StoredValue::SmartContract, applying minor code fixes

@@ -6,13 +6,12 @@ use serde::{Deserialize, Serialize};

#[cfg(test)]
use rand::Rng;
use schemars::JsonSchema;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

In my mind introducing schemars to binary_port is overkill just to save a few lines of code in sidecar so I removed it

… in StoredValue::SmartContract, applying minor code fixes
@zajko zajko force-pushed the execution_engine_changelog branch from 7e13470 to fdb831a Compare December 19, 2024 21:52
@devendran-m devendran-m added the bug Something isn't working label Dec 20, 2024
@EdHastingsCasperAssociation
Copy link
Collaborator

bors r+

Copy link
Contributor

Build succeeded:

@casperlabs-bors-ng casperlabs-bors-ng bot merged commit 7ce0125 into casper-network:feat-2.0 Dec 20, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants