docs: Update package build instructions to use task command.#1394
Conversation
WalkthroughDocumentation updated to remove a standalone Docker image build section and instead add an inline note after the main build instructions stating the build task invokes Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: ASSERTIVE Plan: Pro 📒 Files selected for processing (1)
🧰 Additional context used🧠 Learnings (2)📓 Common learnings📚 Learning: 2025-09-25T19:26:32.436ZApplied to files:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
task package command to task
junhaoliao
left a comment
There was a problem hiding this comment.
for accuracy, the fact we append the user name and a unique hash at the end of the clp-package:dev (e.g., clp-package:dev-junhao-abcd) might be worth mentioning. Though developer can also find out by docker image ls if they really want to run the image manually.
We might simply need to reword as:
... imaged named `clp-package:dev-<user>-<unique-id>`
@quinntaylormitchell what do you think?
if we do not mention that detail, the PR is good to be merged.
for the title, how about:
docs: Update package build instructions to use `task` command.
task package command to tasktask command.
agreed, will do |
Description
The command to build the CLP package is now
taskinstead oftask packageas of PR #1357. This is now reflected in the dev docs. In addition, thetaskcommand runstask docker-images:packageunder the hood, so that is reflected in the dev docs as well.Checklist
breaking change.
Validation performed
Built and served the docs.
Summary by CodeRabbit