Skip to content

feat: support USC fork#15

Open
DylanVerstraete wants to merge 3 commits intomainfrom
feat_usc
Open

feat: support USC fork#15
DylanVerstraete wants to merge 3 commits intomainfrom
feat_usc

Conversation

@DylanVerstraete
Copy link

Supports forking USC network for easy debugging and attestor network bootstrapping

@BradleyOlson64
Copy link

Mind commenting further on when we would use this for testing vs spinning up a local network?

Copy link
Contributor

@atodorov atodorov left a comment

Choose a reason for hiding this comment

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

Couple comments, I think one of them has the potential to break existing creditcoin3 CI jobs.

Also see failing CI jobs.

spec.set_state(
storage_prefix("Sudo", "Key"),
"0xd43593c715fdd31c61141abd04a99fd6822c8558854ccde39a5684e7a56da27d".to_owned(),
);
Copy link
Contributor

Choose a reason for hiding this comment

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

This tool is also used in gluwa/creditcoin3 so removing existing behavior will most likely break things. IMO you should return this back.

src/main.rs Outdated
{
spec.set_state(k, v.clone());
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure if this will continue working with creditcoin3. Looks like it will but we'll have to test it out and if it doesn't need to make more changes.

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.

4 participants