Skip to content

Document logs and logs-watch commands in pluginctl help text#246

Merged
fmartingr merged 1 commit into
masterfrom
pluginctl-logs-help
Jul 13, 2026
Merged

Document logs and logs-watch commands in pluginctl help text#246
fmartingr merged 1 commit into
masterfrom
pluginctl-logs-help

Conversation

@fmartingr

Copy link
Copy Markdown
Contributor

Summary

  • pluginctl's help text listed deploy, disable, enable, and reset, but was missing logs and logs-watch, both of which are already implemented in the command switch.

Test plan

  • Inspected build/pluginctl/main.go to confirm logs/logs-watch are handled and now documented

The pluginctl() switch already handled these subcommands, but they were missing from the usage output.
@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: b7702ebe-82cb-44ba-965e-9f37d71a089b

📥 Commits

Reviewing files that changed from the base of the PR and between 22ef315 and 9ccc2e3.

📒 Files selected for processing (1)
  • build/pluginctl/main.go

📝 Walkthrough

Walkthrough

The helpText usage string in build/pluginctl/main.go is updated to document two additional CLI subcommands: logs <plugin id> and logs-watch <plugin id>.

Changes

Help Text Update

Layer / File(s) Summary
Add logs and logs-watch usage entries
build/pluginctl/main.go
The helpText usage string is extended to include logs <plugin id> and logs-watch <plugin id> command entries.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the main change: adding logs and logs-watch to pluginctl help text.
Description check ✅ Passed The description is directly related and accurately describes the help text update and verification.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch pluginctl-logs-help

Comment @coderabbitai help to get the list of available commands.

@fmartingr
fmartingr merged commit e1f9f88 into master Jul 13, 2026
7 checks passed
@fmartingr
fmartingr deleted the pluginctl-logs-help branch July 13, 2026 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants