[Bug]: indentation clash (formatter vs lintr)
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 1/5
- 見積もり時間
- 1時間未満
- 初心者へのやさしさ
- 92/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- json, r, vscode
調査の方向性
configurationDefaultsから始めて、既存の[r]言語ブロックを見つけます。提案されたエディター設定を使って、デフォルトのタブサイズを2スペースにします。新しいRファイルがデフォルトで2スペースのインデントになり、indentation_linterの警告が出なければ完了です。
索引モデルが issue の本文から書いたものです。
説明
Area
Syntax highlighting
Bug description
VS Code indents with 4 spaces by default, and vscode-R doesn't override this for R. However, lintr expects indentation of 2 spaces. This leads to a harmless, but annoying linting markup for any new R file:
Steps to reproduce
Open a new file and type and write some code with indentation, e.g. a function call over multiple lines:
plot(
1:10
)
Again, the 4 spaces is what gets produced automatically. But lintr will complain that it should only be 2.
[indentation_linter] Indentation should be 2 spaces but is 4 spaces.
Expected behavior
The indentation behaviour or vscode-R should accord with its default linter. My own preference is for that to be 2 spaces, which is the standard that most R linters follow (incl. lintr, air, etc.)
We could set this part of the r block in configurationDefaults.
"[r]": {
"editor.tabSize": 2
}
Environment
vscode-R: 3.2.0
R: 4.6.1
Relevant R package versions
packageVersion('lintr')
[1] ‘3.4.0’
Relevant VS Code settings
Logs and additional context
No response
- 主要言語
- TypeScript
- スター
- 1.2k
- フォーク
- 151
- 平均マージ
- 1日 8時間
- マージ済み PR（30日）
- 76
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
REditorSupport/vscode-R のほかの issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
REditorSupport/vscode-R#1846 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
(feat) Additional Arrow support対応中かも @Fred-Wu が 5 日前に担当しました。 オープンenhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
REditorSupport/vscode-R#1785 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 4/5 3〜5日 初心者へのやさしさ 12/100
REditorSupport/vscode-R#1877 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
Explore a standalone, dependency-free R package for the Data Viewer backend対応中かも @eitsupi が 7 日前に担当しました。 オープン
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
REditorSupport/vscode-R#1860 · コメント 10 件 · リアクション 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 12/100
REditorSupport/vscode-R#1859 ·
メンテナーはふだん 1 日以内に返信
REditorSupport/vscode-R の issue をすべて見る
似ている issue
-
area: repo priority: P2 type: bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
appandflow/stim#4107 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
難易度 1/5 1〜3時間 初心者へのやさしさ 60/100
lingdojo/kana-dojo#32859 · コメント 1 件 · リアクション 5 件 ·
メンテナーはふだん 1 日以内に返信
-
code-quality refactoring
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
github/gh-aw-firewall#10364 ·
メンテナーはふだん 1 日以内に返信
-
Merge the duplicate kaffir_corn record into sorghum対応中かも @Crypto-Shroom が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
Crypto-Shroom/Bird-Feed-Calculator#322 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
drizzle-team/drizzle-orm#6552 ·
メンテナーはふだん 2 日以内に返信