Conversation
NixのLSPとして`nil`と`nixd`のどちらを使うかはホスト環境に任せるため、 devShellから`nil`を削除しました。 セットアップは複雑ではなくプロジェクトのバージョンにも依存しないため、 プロジェクト側で実装を固定する必要はありません。 Claude-Session: https://claude.ai/code/session_01DBicAhQ415nChnMegsrx8r
ncaq
added
the
Type: Dependencies
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
devShellからNixのLSPであるnilを削除するだけの1行変更です。
nilはビルド・チェック・フォーマットのいずれにも関与しておらず、Nixの整形とリントはnixfmt/deadnix/statixが担当したままdevShellに残っています。リポジトリ全体を検索してもnilをツールとして参照している箇所(CIワークフロー、README、スクリプト)はなく、参照切れは発生しません。flake.lockにも変更はありません。
LSPの実装選択をホスト環境に委ねるという方針はPR本文および ncaq/nix-templates#291 と整合しており、妥当な変更だと判断します。品質・ドキュメント・パフォーマンス・セキュリティ・テストの各観点でも指摘事項はありません。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
NixのLSPとして
nilとnixdのどちらを使うかはホスト環境に任せるため、devShellから
nilを削除しました。セットアップは複雑ではなくプロジェクトのバージョンにも依存しないため、
プロジェクト側で実装を固定する必要はありません。
ref ncaq/nix-templates#291