Skip to content

ci: Add CI for Miden examples#228

Merged
huitseeker merged 4 commits into0xMiden:mainfrom
huitseeker:ci-examples
Aug 25, 2025
Merged

ci: Add CI for Miden examples#228
huitseeker merged 4 commits into0xMiden:mainfrom
huitseeker:ci-examples

Conversation

@huitseeker
Copy link
Contributor

@huitseeker huitseeker commented Aug 15, 2025

  • overhaul the benchmarking-cli tool to walk the examples directory, and validate correctness,
  • add CI run options to that CLI,
  • add a CI job that runs those options,
  • add a ./setup-testing.sh job that does basic setup.

Closes #83

Next steps

  • open an issue for porting the examples in old-examples/

@huitseeker huitseeker force-pushed the ci-examples branch 2 times, most recently from ad58bca to 2c1d8f8 Compare August 15, 2025 18:41
- overhaul the benchmarking-cli tool to walk the examples directory, and validate correctness,
- add CI run options to that CLI,
- add a CI job that runs those options,
- add a ./setup-testing.sh job that does basic setup.
@huitseeker huitseeker marked this pull request as draft August 15, 2025 18:58
@huitseeker huitseeker force-pushed the ci-examples branch 2 times, most recently from 32d6677 to baf4848 Compare August 16, 2025 12:24
@huitseeker huitseeker requested a review from Dominik1999 August 16, 2025 12:32
@huitseeker huitseeker marked this pull request as ready for review August 16, 2025 12:32
@Dominik1999
Copy link
Collaborator

This is awesome!

Thanks, was on my list for a long time

Copy link
Collaborator

@Dominik1999 Dominik1999 left a comment

Choose a reason for hiding this comment

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

thanks for the work, I had it on my list for too long and didn't do it. Let me also update the playground itself to the latest VM end of the week.

@@ -0,0 +1,129 @@
name: Test Miden Examples
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is a really good change indeed

@@ -0,0 +1,23 @@
# Older Examples (Miden Assembly 0.7)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is also super useful, let's tackle them one-by-one

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I opened #230

@huitseeker huitseeker merged commit 5eed317 into 0xMiden:main Aug 25, 2025
3 checks passed
@huitseeker huitseeker deleted the ci-examples branch August 25, 2025 14:06
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.

Add a CI to the examples repo

2 participants