-
Notifications
You must be signed in to change notification settings - Fork 134
Hedera-app 0.65 #11921
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
Hedera-app 0.65 #11921
Conversation
Signed-off-by: Kristiyan Selveliev <[email protected]>
Signed-off-by: Kristiyan Selveliev <[email protected]>
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) |
| @Nonnull | ||
| @Override | ||
| public String getServiceName() { | ||
| return super.getStateKey(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not really sure about this
| @Nonnull | ||
| @Override | ||
| public String getServiceName() { | ||
| return "TokenService"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could be improved
| frame = setupInitialFrame(tracerOptions); | ||
|
|
||
| when(worldUpdater.pendingStorageUpdates()).thenReturn(new ArrayList<>()); | ||
| when(worldUpdater.parentUpdater()).thenReturn(Optional.of(rootProxyWorldUpdater)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs to be fixed here
Signed-off-by: Kristiyan Selveliev <[email protected]>
| api("com.graphql-java:graphql-java-extended-scalars:24.0") | ||
| api("com.graphql-java:graphql-java-extended-validation:24.0") | ||
| api("com.hedera.hashgraph:app:0.64.3") | ||
| api("com.hedera.hashgraph:app:0.65.0") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use 0.65.1
|
Closing in favour of #12001 |
Description:
Still in draft
Needs fixes for
java.lang.NoSuchMethodError: 'void com.swirlds.state.spi.WritableKVStateBase.<init>(java.lang.String, java.lang.String)Needs fixes in
OpcodeActionTracerTestNeeds new states:
LAMBDA_STORAGE,EVM_HOOK_STATES- probably just using the defaults will workChecklist