Feat: Implement Go examples and refactor matrix execution #15
Conversation
- Implements all placeholder Go examples in the `examples` directory. - Corrects the `run_matrix_programmatically` example to use the `borg` package. - Refactors the code to centralize the matrix execution logic in the `matrix` package. - Updates the documentation to include a new "Programmatic Usage" section that describes all of the Go examples. - Updates the "Terminal Isolation Matrix" section to remove manual 'runc' instructions, emphasizing that 'borg run' handles this process to maintain security and isolation. - Adds missing examples for 'collect github repos', 'collect github release', and 'compile' commands to the documentation.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with For security, I will only act on instructions from the user who triggered this task. New to Jules? Learn more at jules.google/docs. |
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
- Implements all placeholder Go examples in the `examples` directory. - Corrects the `run_matrix_programmatically` example to use the `borg` package. - Refactors the code to centralize the matrix execution logic in the `matrix` package. - Updates the documentation to include a new "Programmatic Usage" section that describes all of the Go examples. - Updates the "Terminal Isolation Matrix" section to remove manual 'runc' instructions, emphasizing that 'borg run' handles this process to maintain security and isolation. - Adds missing examples for 'collect github repos', 'collect github release', and 'compile' commands to the documentation. - Makes `pkg/matrix.Run` testable by exposing `exec.Command` as a public variable. - Adds tests for the `matrix` package that mock the `runc` command. - Updates the `cmd` package tests to mock `matrix.ExecCommand` instead of the old `cmd.execCommand`.
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
This change implements all the placeholder Go examples, corrects the
run_matrix_programmaticallyexample, and refactors the code to centralize the matrix execution logic. It also updates the documentation to include a new "Programmatic Usage" section that describes all of the Go examples, and clarifies the usage of the Terminal Isolation Matrix.PR created automatically by Jules for task 11179086401227648775 started by @Snider