Skip to content

add '.vscode' to .gitignore#21

Closed
KaiTomotake wants to merge 1 commit into
rust-developers-jp:mainfrom
KaiTomotake:add-ignore-target
Closed

add '.vscode' to .gitignore#21
KaiTomotake wants to merge 1 commit into
rust-developers-jp:mainfrom
KaiTomotake:add-ignore-target

Conversation

@KaiTomotake

Copy link
Copy Markdown
Member

概要

.vsocdeディレクトリを.gitignoreのターゲットに追加。

レビューで見てほしい点

本当に必要な変更かどうか。

@KaiTomotake
KaiTomotake requested a review from hinshiba April 13, 2026 10:19
@hinshiba

Copy link
Copy Markdown
Member

.vscodeは開発環境の統一のためにあってもいいと思いますが,
何が今の.vscodeには入っていますか?

@KaiTomotake

Copy link
Copy Markdown
Member Author
{
    "js/ts.validate.enabled": false
}

TSの警告表示を抑制する設定ファイルです。

@hinshiba

Copy link
Copy Markdown
Member

それは開発者で共有する必要があるので,
共有されるのがいいと思います.

@KaiTomotake

Copy link
Copy Markdown
Member Author

すみません、なぜ共有の必要が?

@hinshiba

Copy link
Copy Markdown
Member

それは何のために設定しているかによりますが,
"js/ts.validate.enabled": falseで組込みの言語サービスを無効化して,外部のlinter等を使うのかと思いまして.
そうであるなら開発環境を統一する意味でも,リポジトリにあったほうがいいのでは?

@KaiTomotake

Copy link
Copy Markdown
Member Author

mdxを編集・追加する際、単純にwarningが邪魔なので追加した次第です。

@hinshiba

Copy link
Copy Markdown
Member

.vscodeは開発の一貫性のためになにかしら共有の必要性が発生する可能性があるので,一般的にはgitignoreに入れないという認識です.
User設定で

{
  "[mdx]": {
    "typescript.validate.enable": false
  }
}

を追加するのはどうでしょうか?

@KaiTomotake

KaiTomotake commented Apr 17, 2026

Copy link
Copy Markdown
Member Author

.vscodeは開発の一貫性のためになにかしら共有の必要性が発生する可能性がある

なるほど・・・。それは確かにそうですね。
まぁどうせ私はTSを使用しないので、とりあえずはUser設定で対応してみましょうか。
また問題が出たらその時考えましょう。
これでいいでしょうか?

@hinshiba

Copy link
Copy Markdown
Member

返信が遅くなって申し訳ない
そうしましょう

@hinshiba hinshiba closed this Apr 23, 2026
@KaiTomotake
KaiTomotake deleted the add-ignore-target branch April 26, 2026 22:57
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.

2 participants