feat(mega-linter-runner): support Apple's native container engine on macOS - #8956
Conversation
…macOS Add "container" to --container-engine (alongside docker/podman) to run MegaLinter with Apple's container CLI on Apple Silicon, without needing Docker Desktop: https://github.com/apple/container Apple's container has no `docker pull`/`docker inspect --format` equivalents, so pull/inspect/logs are branched per engine: - image pull/inspect live under the `image` subcommand - `image inspect` is JSON-only (no --format), parsed for platform match - `logs` uses `-n <n>` instead of `--tail <n>` Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Signed-off-by: Guitton Nicolas <nguitton@cegid.com>
|
@snaquekiller looks good, thanks for the PR :) |
echoix
left a comment
There was a problem hiding this comment.
Looks generally fine, I'm disappointed that Apple took such a generic name, but it's really the name used throughout the project's GitHub page.
…sibling rows Per review from @echoix: the table row was the only one with an inline explanation and URL for one of its allowed values. The detail (Apple Silicon, apple/container link) is already in --help and the usage example, so the table row now just lists the allowed values like the other rows do. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Signed-off-by: Guitton Nicolas <nguitton@cegid.com>
|
Yes, just did — this Mac is Apple Silicon with
Real pull of So the full path (pull → run → cleanup) works end-to-end against the real |
|
Like you suppose i have done everything with AI. i try to do it as clean as possible . i check myself if it's seem ok. For me i didn't saw anything miss. Feel free if you saw anything wrong :) . Im more happy to help and do better. |
|
@snaquekiller that's ok to do everything in AI, there is kind of 1 year i've not written a single line of code :) What makes the differences are the harness around (megalinter has one, starts with /analyze), and how to developer drives the AI and reviews what has been generated :) |
|
@snaquekiller now available with mega-linter-runner@beta :) |
|
tested on my way this morning work well. thanks :) <3 |
|
Great, thanks for the feedback 😊 |
Proposed Changes
containerto--container-engine(docker/podman/container) to run MegaLinter with Apple's native container CLI on Apple Silicon (macOS 15+), without Docker Desktop: https://github.com/apple/containercontainerCLI is not docker-CLI-compatible:image pull/image inspectlive under theimagesubcommand (there is no barepull/inspect)image inspectis JSON-only (no--formatGo template) - parsed for a matchingvariants[].platform.{os,architecture}to keep the "skip pull for pinned versions already available locally" optimizationlogsuses-n <n>instead of--tail <n>--timeoutcleanup path with--container-engine containerRelated to #1160 (podman engine request) - this adds a third engine option for Apple Silicon users who don't want Docker Desktop.
Readiness Checklist
Author/Contributor