Skip to content

Proof lib/fstar support more rbe#1466

Merged
maximebuyse merged 8 commits intomainfrom
proof-lib/fstar-support-more-rbe
Jul 7, 2025
Merged

Proof lib/fstar support more rbe#1466
maximebuyse merged 8 commits intomainfrom
proof-lib/fstar-support-more-rbe

Conversation

@clementblaudeau
Copy link
Copy Markdown
Contributor

This PR extends the proof-lib/fstar representation of rust core and primitives to support more of rust-by-example.
Out of the 24 sets of examples, 19 work. Proper automated testing of those example is left for a future PR.

@clementblaudeau clementblaudeau marked this pull request as ready for review May 16, 2025 15:51
@clementblaudeau clementblaudeau requested a review from a team as a code owner May 16, 2025 15:51
@clementblaudeau clementblaudeau force-pushed the proof-lib/fstar-support-more-rbe branch from 388ea32 to ca0e85e Compare May 16, 2025 22:59
Copy link
Copy Markdown
Contributor

@maximebuyse maximebuyse left a comment

Choose a reason for hiding this comment

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

Looks generally good but I added a few commetns.

Comment thread proof-libs/fstar/core/Core.Fmt.fsti
Comment thread proof-libs/fstar/core/Core.Ops.Function.fsti
Comment thread proof-libs/fstar/core/Core.Str.Iter.fsti Outdated
@clementblaudeau clementblaudeau force-pushed the proof-lib/fstar-support-more-rbe branch from 2412662 to f9e467f Compare June 5, 2025 11:45
@clementblaudeau clementblaudeau force-pushed the proof-lib/fstar-support-more-rbe branch from f9e467f to 88782a0 Compare June 10, 2025 09:57
@clementblaudeau
Copy link
Copy Markdown
Contributor Author

I've rebased the PR on main.

@franziskuskiefer
Copy link
Copy Markdown
Member

@clementblaudeau what's needed to get this merged?

@clementblaudeau
Copy link
Copy Markdown
Contributor Author

Just approval by @maximebuyse I believe.

@franziskuskiefer
Copy link
Copy Markdown
Member

Ok, I re-requested review for you

Copy link
Copy Markdown
Contributor

@maximebuyse maximebuyse left a comment

Choose a reason for hiding this comment

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

Looks good!

@maximebuyse maximebuyse added this pull request to the merge queue Jul 7, 2025
Merged via the queue into main with commit 7df69e3 Jul 7, 2025
18 checks passed
@maximebuyse maximebuyse deleted the proof-lib/fstar-support-more-rbe branch July 7, 2025 09:27
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