Skip to content

use dice-util's new async Attest#65

Merged
iximeow merged 9 commits intomainfrom
ixi/attest-async
Apr 14, 2026
Merged

use dice-util's new async Attest#65
iximeow merged 9 commits intomainfrom
ixi/attest-async

Conversation

@iximeow
Copy link
Copy Markdown
Member

@iximeow iximeow commented Mar 30, 2026

pairs with dice-util#360. this is pretty small in comparison. most of the volume here is because I moved the tests to tokio::test now that they do .awaits, and that means we now have a dev-dependency on tokio.

@augustuswm
Copy link
Copy Markdown

It looks like this was included in the release based on the pin in propolis. Does it make sense to get it merged in to main?

@flihp
Copy link
Copy Markdown
Collaborator

flihp commented Apr 13, 2026

looks like this got left behind in the fray. Same w/ oxidecomputer/dice-util#360. the dice-verifier changes were approved ~2 weeks ago so the merge button just didn't get pressed. I can do that easy enough. Will circle back on this one once that builds green.

@iximeow
Copy link
Copy Markdown
Member Author

iximeow commented Apr 14, 2026

clicked the button on dice-util#360 and bumped to the now-green commit with that over there: oxidecomputer/dice-util@d7472bf

@iximeow iximeow merged commit eb9e80d into main Apr 14, 2026
11 checks passed
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