Repository navigation
Conversation
|
@codex review |
There was a problem hiding this comment.
Pull request overview
Adds a new developer-guide documentation page describing OpenRemote’s CI/CD pipeline concepts introduced during the pipeline migration referenced in openremote/core#2696, addressing the documentation request in Issue #161.
Changes:
- Introduces a new
CI/CD Pipelinedeveloper-guide page covering pipeline modification/testing, workflow structure, artifacts, Gradle vs workflow responsibilities, and best/worst practices. - Documents optimization strategies (detect-changes, minimal dependency installs, Git LFS usage, artifact reuse).
- Summarizes GitHub Actions concepts relevant to the pipeline (composite actions, reusable workflows, conditional expressions).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5ef00fa8ea
ℹ️ 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".
c4d762a to
b9ec506
Compare
MartinaeyNL
left a comment
There was a problem hiding this comment.
⚠️ Initial review by an AI agent ⚠️
Read through it against the actual workflows — checks, skip inputs, detect-changes outputs, sharding + UI App Tests Status, the tsbuildinfo trick, type=gha, the or-manager-build exception, deploy in .ci_cd/ci_cd.json, and the IS_MAIN_REPO/openremote/.ci_cd compat note all match the code, and all of #161's acceptance criteria are covered. Only open point is the merge-order dependency on openremote/core#3448 (inline at the Custom Projects section).
Adds a developer guide page for the CI/CD pipeline of the core repository. It covers the workflows introduced in the pipeline migration (openremote/core#2696) and the custom project workflows (openremote/core#3441).
Structure
ci_cd.yml