Skip to content

minimal-crates: blacksmith runners & noq support - #10

Merged
flub merged 7 commits into
mainfrom
flub/blacksmith-runners
Oct 28, 2026
Merged

flub merged 7 commits into
mainfrom
flub/blacksmith-runners

Conversation

@flub

@flub flub commented Oct 23, 2026 •

Copy link
Copy Markdown
Collaborator

Updates to minimal-crates workflow:

  • Use blacksmith runners
  • Support noq by:
    • having an option to install more build deps needed for awl-lc-rs FIPS support
    • allow customising the tested creates, to exclude noq fuzz crate

@n0bot n0bot Bot added this to iroh Oct 23, 2026
@github-project-automation github-project-automation Bot moved this to 🚑 Needs Triage in iroh Oct 23, 2026
@flub flub changed the title Use blacksmith runners for minimal-crates workflow minimal-crates: blacksmith runners & noq support Oct 23, 2026
@flub
flub marked this pull request as ready for review October 23, 2026 14:01
@flub
flub requested a review from Arqu October 23, 2026 14:01
Comment thread .github/workflows/minimal-crates.yaml Outdated

# Blacksmith's Windows image ships no pre-installed MSYS2 (C:\msys64).
release: true
# - name: Install Go (required for aws-lc-rs FIPS)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we drop this if its not needed here

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

oh yes, forgot to clean up. thanks

@Arqu Arqu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a nit

@flub
flub merged commit a52bc33 into main Oct 28, 2026
2 checks passed
@flub
flub deleted the flub/blacksmith-runners branch October 28, 2026 11:25
@github-project-automation github-project-automation Bot moved this from 🚑 Needs Triage to ✅ Done in iroh Oct 28, 2026
Stanley00 pushed a commit to stanley-fork/noq that referenced this pull request Oct 28, 2026
## Description

This moves the minimal-crates tests to the shared workflow. This will
allow us to make this test required again as it should no longer fail
when we release our own crates.

Also triggers the wine and zizmor workflows for the merge queue, so that
we can make those required checks as well.

## API Changes

n/a

## Notes & open questions

Needs a new tag on the workflows repo once
n0-computer/workflows#10 is merged. Then pinact
and zizmor will be happy here.

This will allow us making the min-crates a required check again.

## Change checklist

- [x] Self-review.
- [x] Documentation updates following the [style
guide](https://rust-lang.github.io/rfcs/1574-more-api-documentation-conventions.html#appendix-a-full-conventions-text),
if relevant.
- [x] Tests if relevant.
- [x] All API changes documented.
- [x] `cargo make` passes locally.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants