Skip to content

fix lint#1166

Merged
pirosiki197 merged 1 commit intomainfrom
fix/lint
Mar 1, 2026
Merged

fix lint#1166
pirosiki197 merged 1 commit intomainfrom
fix/lint

Conversation

@pirosiki197
Copy link
Copy Markdown
Contributor

@pirosiki197 pirosiki197 commented Mar 1, 2026

なぜやるか

lintのCIが落ちてる

やったこと

直した

やらなかったこと

資料

Summary by CodeRabbit

リリースノート

  • リファクタリング (Refactor)
    • 内部の文字列組立処理を統一してコードの一貫性と保守性を向上しました。出力されるテンプレートやビルド定義の内容は変わらず、ユーザー向け機能や公開APIに影響はありません。

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 1, 2026

Preview (prod backend + PR dashboard) → https://1166.ns-preview.trapti.tech/

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 52c01d5 and f83b2bd.

📒 Files selected for processing (2)
  • pkg/infrastructure/staticserver/caddy/server.go
  • pkg/usecase/builder/build_buildkit.go
🚧 Files skipped from review as they are similar to previous changes (1)
  • pkg/usecase/builder/build_buildkit.go

📝 Walkthrough

Walkthrough

2つのファイルで、文字列生成時の書き込みをWriteString(fmt.Sprintf(...))からfmt.Fprintf(...)へ置き換え、ビルダー/バッファへの書き込み手法を統一。出力内容や制御フローに変更はない。

Changes

Cohort / File(s) Summary
書き込み手法の統一
pkg/infrastructure/staticserver/caddy/server.go, pkg/usecase/builder/build_buildkit.go
複数箇所でWriteString(fmt.Sprintf(...))dockerfile.WriteString(fmt.Sprintf(...))fmt.Fprintf(&...)に置換。テンプレート/出力文字列は維持し、初期化や公開APIの変更はなし。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive プルリクエストのタイトルは「fix lint」と非常に曖昧で、何を修正したのか具体的に説明していません。コード変更の内容(fmt.Sprintfの使用をfmt.Fprintfに置き換える最適化)と一致していません。 タイトルをより具体的に「Replace fmt.Sprintf with fmt.Fprintf for better efficiency」のような形に変更してください。
Description check ❓ Inconclusive プルリクエストの説明は極めて簡潔で、「lintのCIが落ちてる」「直した」という最小限の記述のみです。テンプレートの形式は守られていますが、実装内容の詳細が不足しており、レビュアーが変更の目的と内容を理解するのに十分な情報がありません。 具体的なlint違反の内容、修正した理由、影響範囲などの詳細を説明セクションに追加してください。
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/lint

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.

❤️ Share

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

@pirosiki197 pirosiki197 enabled auto-merge March 1, 2026 13:59
@pirosiki197 pirosiki197 merged commit 6f4738b into main Mar 1, 2026
15 checks passed
@pirosiki197 pirosiki197 deleted the fix/lint branch March 1, 2026 14:02
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.

1 participant