Skip to content

Add local development support for setup and proof runs#196

Merged
automergerpr-permission-manager[bot] merged 3 commits into
proofsfrom
mk/local-dev
May 20, 2026
Merged

Add local development support for setup and proof runs#196
automergerpr-permission-manager[bot] merged 3 commits into
proofsfrom
mk/local-dev

Conversation

@mariaKt
Copy link
Copy Markdown

@mariaKt mariaKt commented May 19, 2026

This PR re-enables local development after the switch from the mir-semantics submodule to kompass.

  • setup.sh --local-dev: install kmir and kompass from local paths (via KMIR_LOCAL and KOMPASS_LOCAL env vars)
    instead of pinned remote version
  • run-proofs.sh: detect local-dev mode and include local commit hashes in proof directory names"

I also added a clean target for simpler rebuilding, since it would occur often for local development.

@mariaKt mariaKt requested a review from dkcumming May 19, 2026 23:02
@mariaKt mariaKt marked this pull request as ready for review May 19, 2026 23:02
Copy link
Copy Markdown
Collaborator

@dkcumming dkcumming left a comment

Choose a reason for hiding this comment

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

Nice, I don't have a chance to run it but it looks right!

@mariaKt
Copy link
Copy Markdown
Author

mariaKt commented May 20, 2026

I tested it locally myself

  • with sequences of local builds in different commits to test the naming, and sometimes with errors so that they would fail, to make sure that the changes were picked up.
  • local and non local, to test the use of the marker.

@automergerpr-permission-manager automergerpr-permission-manager Bot merged commit 1432f48 into proofs May 20, 2026
12 checks passed
@mariaKt mariaKt deleted the mk/local-dev branch May 20, 2026 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants