Skip to content

Verify pkg/slayers/path/epic#9

Merged
henriman merged 10 commits into
sif-io-specfrom
sif-pkg-slayers-path-epic
Sep 20, 2025
Merged

Verify pkg/slayers/path/epic#9
henriman merged 10 commits into
sif-io-specfrom
sif-pkg-slayers-path-epic

Conversation

@henriman

@henriman henriman commented Aug 6, 2025

Copy link
Copy Markdown
Owner

Verifying pkg/slayers/path/epic necessitated adding annotations to pkg/slayers/path/scion which has not been verified yet, as the package itself has not been verified yet. The other modified (and previously verified) packages verify successfully.

Issues #14 and #15 have been opened to keep track of the assumptions still present here.

@henriman
henriman marked this pull request as ready for review August 18, 2025 18:26
@henriman
henriman requested a review from jcp19 August 18, 2025 18:26
@henriman

Copy link
Copy Markdown
Owner Author

Given the remaining issues with fix-846, for now I will also use PR #10 to implement hyper functions for pkg/slayers/path/epic

@jcp19 jcp19 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, except for the TODOs for later. What's keeping us from merging this PR atm?

@henriman
henriman changed the base branch from sif-pkg-slayers-path to sif-io-spec September 20, 2025 13:53
@henriman
henriman merged commit 7eaabb1 into sif-io-spec Sep 20, 2025
@henriman
henriman deleted the sif-pkg-slayers-path-epic branch October 7, 2025 09:40
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.

2 participants