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.

Sign up for GitHub

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

Jump to bottom

Add Scope Interface #1826

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add Scope Interface #1826

wants to merge 1 commit into from
+95 −267

Conversation

RodrigoVillar
Copy link
Contributor

This PR modifies the tstate package by changing scope from simply being state.Keys to an interface which can be used to pass in state.Keys or another type (e.g. SimulatedKeys).

This PR also removes the tstate recorder.

@RodrigoVillar RodrigoVillar marked this pull request as ready for review December 3, 2024 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aaronbuchwald aaronbuchwald Awaiting requested review from aaronbuchwald aaronbuchwald is a code owner

@joshua-kim joshua-kim Awaiting requested review from joshua-kim joshua-kim is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@RodrigoVillar