Skip to content

fix(cmd/feature): lowercase log strings - #1008

Merged
skevetter merged 1 commit into
mainfrom
cmd-reviewer/feature-lowercase-logs
Aug 14, 2026
Merged

skevetter merged 1 commit into
mainfrom
cmd-reviewer/feature-lowercase-logs

Conversation

@devsy-app

@devsy-app devsy-app Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

cmd/ subdirectory reviewed

cmd/feature — the devsy feature subcommand directory.

Issue found

Three log strings used capitalized leading text, violating the repo's logging convention (lowercase logs, per AGENTS.md):

  • cmd/feature/publish.go:93 — "Publishing feature %q"
  • cmd/feature/publish.go:106 — "Feature published successfully: %s"
  • cmd/feature/upgrade.go:139 — "Upgraded %s: %s to %s"

Minimal change

Lowercased the three log strings (no behavior or flag changes):

  • publish.go — "publishing feature %q" and "feature published successfully: %s"
  • upgrade.go — "upgraded %s: %s to %s"

Diff: 2 files changed, 3 insertions(+), 3 deletions(-).

Verification

  • task cli:format — clean
  • task cli:lint:ci — 0 issues
  • task cli:test — cmd/feature tests pass; only pkg/git (TestRepoClone*) fails, which is a known pre-existing failure on origin/main unrelated to this change (this change does not touch pkg/git)

This PR was created by an AI agent as part of an automated daily CLI review job.

Convert three capitalized log strings in cmd/feature to lowercase per the repo lowercase-logs convention. publish.go (Publishing/Feature published successfully) and upgrade.go (Upgraded).
@netlify

netlify Bot commented Aug 12, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for devsydev canceled.

Name Link
🔨 Latest commit df2b0da
🔍 Latest deploy log https://app.netlify.com/projects/devsydev/deploys/6a7c414c6f236100080a0acf

@netlify

netlify Bot commented Aug 12, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for images-devsy-sh canceled.

Name Link
🔨 Latest commit df2b0da
🔍 Latest deploy log https://app.netlify.com/projects/images-devsy-sh/deploys/6a7c414cad46be000863a9c9

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

AI Reviewer: run a review on demand. To trigger the first review automatically, go to your organization or repository integration settings. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

@skevetter
skevetter marked this pull request as ready for review August 14, 2026 02:29
@skevetter
skevetter merged commit d653b93 into main Aug 14, 2026
163 of 167 checks passed
@skevetter
skevetter deleted the cmd-reviewer/feature-lowercase-logs branch August 14, 2026 03:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant