Skip to content

add '.vscode' to .gitignore#21

Closed
KaiTomotake wants to merge 1 commit intoRust-Developers-JP:mainfrom
KaiTomotake:add-ignore-target
Closed

add '.vscode' to .gitignore#21
KaiTomotake wants to merge 1 commit intoRust-Developers-JP:mainfrom
KaiTomotake:add-ignore-target

Conversation

@KaiTomotake
Copy link
Copy Markdown
Member

概要

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

レビューで見てほしい点

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

@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
Copy link
Copy Markdown
Member Author

KaiTomotake commented Apr 17, 2026

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

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

@hinshiba
Copy link
Copy Markdown
Member

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

@hinshiba hinshiba closed this Apr 23, 2026
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Organization Tasks 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

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants