Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🔥 remove .golangci.yml #586

Merged
merged 1 commit into from
Mar 11, 2025
Merged

🔥 remove .golangci.yml #586

merged 1 commit into from
Mar 11, 2025

Conversation

iChemy
Copy link
Contributor

@iChemy iChemy commented Mar 3, 2025

No description provided.

@iChemy
Copy link
Contributor Author

iChemy commented Mar 3, 2025

golangci-lint は 設定ファイルとして .golangci.yml を参照するがこれのフォーマットがどうやら正しくなく ci で lint が fail していた.

変更前の .golangci.yml 自体には何も書き込まれてなかったため,これを削除して デフォルトの設定で golangci-lint が動作するようにした.

ここら辺が参考になりそう(古いけど)
https://tech.yyh-gl.dev/blog/golangci-lint-custom-settings/

@iChemy
Copy link
Contributor Author

iChemy commented Mar 3, 2025

もし,デフォルト設定が望ましくないなら .golangci.yml に詳細な設定を書くのはあり.(issue を生やすなり,別のプルリクで対応するのがよさそう.)

@iChemy
Copy link
Contributor Author

iChemy commented Mar 3, 2025

traQ や traPortfolio などの設定ファイルを真似てみるのはありかもね.

@ras0q
Copy link
Member

ras0q commented Mar 3, 2025

空の.golangci.ymlを置いてたのはグローバルに~/.golangci.ymlがあった時にそっちが使われてプロジェクト内でスタイルが分かれるからですね
なぜ急にgolangci-lintが空ファイルを拒否するようになったのかは分からないですが、グローバルに置いてる人も少ないと思うのでとりあえず消してCIをPASSさせるのは良いと思います

@iChemy
Copy link
Contributor Author

iChemy commented Mar 3, 2025

なるほど,そういう意図があったんですね...
ローカルでは lint にかけず push すて github actions の方で結果だけ見てたので,(よくない)
lint のローカルでの環境の差異は気にしてませんでした,

一旦 ファイルを削除する方向で調整したいと思います
ありがとうございます🙇!!

@ras0q
Copy link
Member

ras0q commented Mar 3, 2025

tipsとして例えばVSCodeではgoのフォーマッターをgolangci-lintにする設定があるのでこれをformatOnSaveと一緒に使うと常にフォーマットが整ってる状態になってgoodになります

@iChemy iChemy requested a review from Nzt3-gh March 4, 2025 12:23
Copy link
Contributor

@Nzt3-gh Nzt3-gh left a comment

Choose a reason for hiding this comment

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

現在 .golangci.yml が空であることがフォーマット不備として扱われ golangci-lint が機能せず CI が通らないこと、該当ファイルを削除することで適切にCIが機能することを確認しました。

@iChemy iChemy merged commit 6ff4ed7 into main Mar 11, 2025
6 checks passed
@iChemy iChemy deleted the fix-lint branch March 11, 2025 14:17
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.

3 participants