Improve remote-ci compatibility and generalize docs#12
Conversation
There was a problem hiding this comment.
Pull request overview
This PR improves the robustness and real-world compatibility of the ci-self remote workflows (notably remote-ci) by adding SSH identity support, better sync defaults, and safer bootstrap/artifact-collection behavior, alongside generalized documentation and a new English README.
Changes:
- Added
-i/--identitysupport across remote commands, improved remote path handling, and added SSH fallback paths (remoteci-selflocation + artifact fetch). - Hardened
remote-cisync behavior (generated-directory excludes, optional.git/sync, rsync version fallback, prefer Homebrew rsync on macOS). - Updated and generalized documentation examples and added
README_EN.md.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
ops/ci/ci_self.sh |
Adds identity-aware SSH/rsync execution, improves remote-ci sync/bootstrap/fetch behavior, and generalizes examples/config. |
ops/ci/ci_self_test.go |
Expands test coverage for new remote-ci/remote-* behaviors (identity, exclusions, fallbacks, bootstrap skipping). |
docs/ci/QUICKSTART.md |
Generalizes remote-ci guidance and documents identity usage and sync/bootstrap behavior. |
README.md |
Generalizes repository-specific examples and documents updated remote-ci behavior and options. |
README_EN.md |
Adds an English overview and quickstart for local + remote-ci usage and assumptions. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 605e55af8a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Summary
remote-cifor real-world SSH/bootstrap usageREADME_EN.mdWhat Changed
-i/--identitysupport and robust remote path handling forremote-cirsyncon macOS when available~/.local/bin/ci-selfon the remote machine whenci-selfis not in PATHgh authis missing instead of surfacing a noisy warningVerification
bash -n ops/ci/ci_self.shmise x -- go test ./...remote-civerification from Machine B to Machine A with result collection