Skip to content

Conversation

@newt239
Copy link
Member

@newt239 newt239 commented Nov 20, 2025

close #229
close #139

概要

golangci-lintの導入と、lintツールによる指摘事項の修正を行いました。

変更内容

1. CI/CDの追加

  • .github/workflows/lint.ymlを追加し、golangci-lintによる自動リントを実装
  • masterブランチへのpush/PR時に自動実行

2. Goバージョンアップ

  • go.mod: Go 1.19 → 1.20にアップグレード(Dockerで指定しているイメージのバージョンに統一しました)

3. エラーハンドリングの改善

以下のlintツール(errcheck、staticcheck)の指摘事項を修正:

  • JWTトークン生成時のエラーチェック追加 (pkg/api/authenticator/authenticator.go)
    • jwt.Set()の戻り値をチェック
  • リクエストボディのバインド時のエラーチェック追加
  • トランザクションのロールバック時のエラーログ出力
  • データベース操作の戻り値チェック追加
    • pkg/batch/mattermost_post.go: Exec()の戻り値をチェック

4. 非推奨APIの置き換え

  • pkg/mattermost/post_mattermost_create_user.go: ioutil.ReadAllio.ReadAllに変更
  • ioutil.ReadAllio.ReadAllは互換性があります

@newt239 newt239 self-assigned this Nov 20, 2025
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 20, 2025

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
spec-core 168180d Commit Preview URL

Branch Preview URL
Nov 22 2025, 02:04 AM

@newt239 newt239 requested a review from Copilot November 22, 2025 02:12
Copilot finished reviewing on behalf of newt239 November 22, 2025 02:14
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

このPRは、golangci-lintの導入とそれに伴うコード品質の改善を目的としています。具体的には、Go 1.19から1.20へのバージョンアップ、errcheckやstaticcheckによる指摘事項の修正、非推奨APIの置き換えを実施しています。

  • CI/CDパイプラインにgolangci-lintによる自動リントを追加
  • エラーハンドリングの強化(JWT生成、リクエストバインド、トランザクション、データベース操作)
  • 非推奨のioutil.ReadAllio.ReadAllに置き換え

Reviewed changes

Copilot reviewed 37 out of 37 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/workflows/lint.yml golangci-lintの自動実行ワークフローを追加
go.mod Goバージョンを1.19から1.20にアップグレード
pkg/mattermost/post_mattermost_create_user.go 非推奨のioutil.ReadAllio.ReadAllに変更
pkg/batch/mattermost_post.go データベース操作の戻り値チェックとエラーログ出力を追加
pkg/api/authenticator/authenticator.go JWTトークン生成時のjwt.Set()エラーチェックを追加、変数名の修正
pkg/api/server/put_*.go リクエストボディのバインドエラーチェックとトランザクションロールバックのエラーログ出力を追加(16ファイル)
pkg/api/server/post_*.go リクエストボディのバインドエラーチェックとトランザクションロールバックのエラーログ出力を追加(10ファイル)
pkg/api/server/delete_*.go トランザクションロールバックのエラーログ出力を追加(5ファイル)
pkg/api/server/irregular/post_mattermost_cmd.go リクエストボディのバインドエラーチェックとトランザクションロールバックのエラーログ出力を追加

@newt239 newt239 requested a review from saka-naname November 23, 2025 13:56
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.

golang-cliの導入 io/ioutilの非推奨化に対する対応

2 participants