Closed rsrohitsingh682 closed 3 weeks ago
We should add unit tests as well. That the correct SSH CLI is constructed. We don't have to run nix copy
in the tests.
I'm doing some refactor on main
. Don't worry about rebasing just yet (until this PR is ready and reviewed).
@rsrohitsingh682 main
has been refactored. You can build on top of it. It probably is easier to just reapply your changes on top. You can start from
https://github.com/juspay/omnix/blob/main/crates/nixci/src/command/build.rs
Heads up: om ci build
is now om ci run
.
You can find the command entry point in, https://github.com/juspay/omnix/blob/main/crates/nixci/src/command/run.rs
CI failed because docs are not present for the files that I added. Will add those tomorrow.
I need to fix the doc targets, but meanwhile you can use:
cargo-watch -x cargo doc -p nixci
(Zed or VSCode should also show these as warnings)
Alright, cheers. Let me take over from here.
Resolves #188
You can run om ci like:
om ci build --on user@ip-address
Using nix
nix run github:juspay/omnix/om-ci-remote-build -- ci run --on user@ip