Skip to content

chore(examples): reserve extraction folder for auto-generated files#1754

Merged
clementblaudeau merged 3 commits intomainfrom
alex/only-extraction-in-extraction
Nov 20, 2025
Merged

chore(examples): reserve extraction folder for auto-generated files#1754
clementblaudeau merged 3 commits intomainfrom
alex/only-extraction-in-extraction

Conversation

@abentkamp
Copy link
Copy Markdown
Contributor

As discussed with @karthikbhargavan, only extracted files should be put into the extraction folder. So I started to enforce that policy on the examples in the example folder.

Additional notes:

  • I haven't touched coq-example because I wasn't able to run it.
  • I added some .gitignores to ignore files produced when running the examples.
  • I added the hax library to the lakefile of barrett because this was needed to make the example work in the first place.

@abentkamp abentkamp requested a review from a team as a code owner November 5, 2025 10:04
@abentkamp abentkamp force-pushed the alex/only-extraction-in-extraction branch from a720c5c to 751d1f3 Compare November 5, 2025 12:26
@abentkamp
Copy link
Copy Markdown
Contributor Author

In this morning's meeting we decided to make this change only to Lean examples for now. The Makefile for F* will not look into subfolders, which makes it harder to move.

@abentkamp abentkamp self-assigned this Nov 6, 2025
@abentkamp abentkamp force-pushed the alex/only-extraction-in-extraction branch from 751d1f3 to 17b63ba Compare November 6, 2025 12:18
Copy link
Copy Markdown
Contributor

@clementblaudeau clementblaudeau left a comment

Choose a reason for hiding this comment

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

I agree with the general philosophy of having lean config in the /proof/lean directory instead of /proofs/lean/extraction. Thanks for the change!

@clementblaudeau clementblaudeau added this pull request to the merge queue Nov 20, 2025
Merged via the queue into main with commit 8bf86b4 Nov 20, 2025
17 of 18 checks passed
@clementblaudeau clementblaudeau deleted the alex/only-extraction-in-extraction branch November 20, 2025 09:42
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