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

Another implemaentation of GetPublicStatesInfo on InputsSignals instead of on PubSignals #90

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

olomix
Copy link
Collaborator

@olomix olomix commented Jan 7, 2025

No description provided.

@olomix olomix requested a review from demonsh January 7, 2025 11:22
@demonsh demonsh requested a review from vmidyllic January 9, 2025 10:35
@olomix olomix changed the title Another implemaentation of GetStatsInfo on InputsSignals instead of on PubSignals Another implemaentation of GetPublicStatesInfo on InputsSignals instead of on PubSignals Jan 14, 2025
State *merkletree.Hash
ID core.ID `json:"id"`
State merkletree.Hash `json:"state"`
}
Copy link
Contributor

Choose a reason for hiding this comment

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

this is breaking change, isn't it?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This struct was added in master branch only. It is not versioned yet.

@olomix olomix requested a review from vmidyllic January 15, 2025 10:47
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