Repository navigation
Align lint and logs CLI options - #75265
Conversation
Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
|
β PR Specification Reviewer completed successfully! PR Specification ReviewerOutcome: The agent completed and reported no action. A no-op output alone does not verify that all required work was completed or that publication was possible. Agent messages are omitted from this comment. The category uses structured run status, not interpretation of agent prose.
|
|
β Test Quality Sentinel completed test quality analysis. Test Quality SentinelOutcome: The agent completed and reported no action. A no-op output alone does not verify that all required work was completed or that publication was possible. Agent messages are omitted from this comment. The category uses structured run status, not interpretation of agent prose.
|
|
β Ponytail Reviewer completed successfully! Ponytail ReviewerOutcome: The agent completed and reported no action. A no-op output alone does not verify that all required work was completed or that publication was possible. Agent messages are omitted from this comment. The category uses structured run status, not interpretation of agent prose.
|
|
β PR Code Quality Reviewer completed the code quality review.
|
|
π§ Matt Pocock Skills Reviewer has completed the skills-based review. β
|
|
β Design Decision Gate ποΈ completed the design decision gate check. See the comment below for the result and any generated ADR draft. Design Decision Gate ποΈOutcome: The agent completed and reported no action. A no-op output alone does not verify that all required work was completed or that publication was possible. Agent messages are omitted from this comment. The category uses structured run status, not interpretation of agent prose.
|
There was a problem hiding this comment.
π’ Approval recommended
The reviewed changes align the CLI behavior and documentation, with a regression test covering the new lint shorthand and no active issues identified.
No open issues π
What changed in this PR
Updates the lint and logs CLI interfaces and documentation to keep available options and references aligned.
Changes:
- Adds the
-jshorthand forlint --json. - Adds a regression test for the new shorthand.
- Documents the
logs--episodes,--outcomes, and--outcomes-diroptions.
| File | Description |
|---|---|
pkg/βcli/βlint_command.go |
Registers -j as the shorthand for --json. |
pkg/βcli/βlint_command_test.go |
Verifies the new lint JSON shorthand. |
docs/βsrc/βcontent/βdocs/βsetup/βcli.md |
Updates lint and logs CLI option documentation. |
Review effort: Lite (auto)
|
π This pull request is included in a new release. Release: |
The CLI reference omitted three
logsoptions, andlint --jsonlacked the-jshorthand used by other commands.-jfor--jsonand update its reference documentation.--episodes,--outcomes, and--outcomes-dirin the main options list and--outcomesin the outcomes section.