@adidahiya @ nchen63によるこのコメントを参照してください。

@ nchen63から：

タブ-> xスペースおよびxスペース->タブは修正されますが、xスペース-> yスペースは修正されません。

x spacesからy spacesがあります。

私のプロジェクトは2つと4つのスペースの両方を使用しています。 ルールx spacesをy spaces修正すると非常に便利です。

また、サイズ違反を修正するか、少なくともエラーとして報告して修正できるようにすることを期待して、この問題に遭遇しました。 現時点では、サイト（https://palantir.github.io/tslint/rules/indent/）に記載されているインデントサイズの構成は実際には何もしていないようです。

はい、このバグを修正してください。 Angular CLIは、ファイルまたはプロジェクトを生成するときにインデントレベルに2つのスペースを使用し、すべての引用符は一重引用符です。 次に、tslintを実行して、ユーザーのtslint.jsonに従ってこれらを修正します。 引用符はうまく機能しますが（私の好みに応じて二重引用符に変換されます）、インデントは2スペースのままです（私は4スペースを好みます）。 Tslintは、実際のTAB文字を検出した場合にのみエラーを報告しますが、スペースの数もチェックする必要があると考えられます。

https://github.com/palantir/tslint/blob/master/src/rules/indentRule.tsで使用されている実装は非常に素朴で、 x spaces -> y spaces機能するとは思わないようです

たとえば、2スペースのインデント幅が選択されたとし、次のコードを検討します。

foo = {
    a: {
  b: {
    c: 'c'
  }
    },
    d: 'd'
}

現在の実装では、これがまったく失敗することをどのようにして知ることができますか？ スペースの各シーケンスは正規表現/ /渡します。 2つのスペースの倍数を探しても、これは合格です。 同様に、マルチレベルのインデントシナリオでは、4つの先頭のスペースが合格シナリオになる可能性がありますが、4つのスペースで単一レベルのインデントを開始すると失敗するはずです。

eslintと同様に、どのソリューションでもASTをトラバースする必要があると思います（https://github.com/eslint/eslint/blob/master/lib/rules/indent.js）。 その欠点は、 tabs -> spacesまたはspaces -> tabsパフォーマンスがわずかに低下することです。 設定に基づいて実装を選択することで回避できますが、タブとスペースを組み合わせて使用​​すると、現在の実装は失敗することが予想されます。その場合は、ASTベースのソリューションのみを使用する必要があります。

eslintの実装で行われるAST作業の少なくとも一部は、typescriptのライブラリで直接処理できると思います。したがって、ソリューションを作成するのはそれほど難しくないはずです。

現在の実装では、これがまったく失敗することをどのようにして知ることができますか？

_技術的に_する必要はありません。 alignルールを使用すると、そのようなコードを記述できなくなります。 もちろん、 indentルールでもASTを歩くことができます。 私が気にしているのは、 indentとalign組み合わせによって、問題の説明に書いたことを達成できるということだけです。

整列ルールは、そのようなコードを書くことを防ぎます。

良い！ 私がソースコードを調べていたとき、私はそれを見逃しました。 あそこもピークを迎えます。 TBH、私は既存のルールを活用して他の人をできるだけ素朴に保つというアイデアが好きです。

同じ問題が発生しています。 以下のルールは、スペースの代わりに使用されているタブをキャッチしますが、誤った数のスペースをキャッチしません。 2を他の数値に変更できますが、それでもエラーは発生しません。 tslint5.5.0を使用しています。

"indent": [true, "spaces", 2],

これに関する更新はありますか？ （誰かがまだ修正に取り組んでいない場合、私はそれを試してみるつもりです。）

@mDibyoはそれのために行きます

@mDibyoこれについて何か進歩はありましたか？

部分的な実装があります。 それを完了し、週末にPRを立てようとします

プロジェクトでtslint-plugin-prettierを使用してよりきれいなフォーマットを使用し始めて以来、この問題の優先度が低くなっていることをたいと思います。 将来的には、組み込みの構成にprettierの公式サポートを追加し、それに応じて、フォーマットに関連しないコアルールにフォーカスを調整しようとする可能性があります。

一方、Prettierは、ほとんどのコードをすべての行に収めるように努めています。

あなたがそれを望まない場合はどうなりますか？

印刷幅を120に設定すると、よりきれいに、過度にコンパクトな、または望ましくないコードが生成される可能性があります。

プロジェクトでは120文字を使用しています。

また、それはさらに別の依存関係です。 通常のTSLintルールを使用したいと思います。

通常のTSLintルールを使用したいと思います。

@ glen-84ええ、それで問題ありません。そのため、TSLintからすべてのフォーマット規則を削除し、外部フォーマッターに完全に委任する必要があると言っているわけではありません。 Prettierは明らかに意見があり、誰もがそれを使用することを選択するわけではありません。 この問題はまだPRに開かれています。

この問題はどうなっているのですか？

私の広報は進行中ですが、多くの場合、まだ時間が必要です。 @cyberhck

ですから、すぐにマージされると予想しています：slightly_smiling_face：

ここに更新はありますか？

おそらく、 typescript-formatterのメンテナにいくつかのAPIを公開するように依頼することができますか？ パッケージは、ソースファイル全体がtslint.jsonファイルの設定に従ってフォーマットます。

ただし、TSLintのフィクサー設定をうまく活用するには助けが必要な場合があります。

問題は、インデントサイズがオフの場合ではなく、間違った空白文字を使用した場合にのみ失敗が報告されることであるようです。

@adidahiya間違ったインデント文字が使用されていると、エラーを報告できません。 ルールをspaces / 4に設定し、次のようなものがある場合：

export function foo() {
return 123;
}

また

export function foo() {
<tab>return 123;
}

エラーは報告されません。 元のコメントによると、空白文字が間違っている場合は報告されますか？

これについて何か進歩はありますか？ ただ尋ねる😝

助言？ Prettierを使用します。

誰かがtslint-eslint-rulesを試し

@jscharett tslint-eslint-rulesは、 ter-indentルールで機能します。 残念ながら、JSXインデントはカバーされていません...

ここで修正の希望はありますか？

このバグはv5.10.0ではまだ修正されていません

TSLintがPrettierと同様にJSコードをフォーマットできるとは思いません。 これは複雑な問題であり、Prettierは他の誰よりもうまく解決しました。 特にプロジェクトでは両方のツールを同時に使用することが多く、競合が発生する可能性があるため、これをTSLintに依存するべきではないと思います...

編集：この問題がどのように複雑であるかをよりよく理解するには、このPRを確認するか、Prettierソースコードを確認してください。 これらのデモンストレーションが「複雑」だと思わない場合は、PRでこのプロジェクトを支援してください😄！

@aervin私はここで反対する傾向があります。 私の意見では、2つのプロジェクトの目的は異なります。 TSLintは検証とより一致しているのに対し、Prettierはフォーマットのカテゴリに分類されます。 はい、TSLintはいくつかのルールのフォーマットを行うことができますが、リンターとしての意図された目的は検証に戻ります。

Prettierに依存することの問題は、それが意見を述べられていることです。 そのスタイルに同意するのは素晴らしいことですが、同意しない場合はどうなりますか？ 以前はみんなJSLintを使っていましたが、それがとても意見が多かったのでみんな不平を言いました。 次に、JSHintとJSCSが登場し、ある程度の制御が可能になりました。 これで、 @ eslintのような強力なツールができ、プラグアンドプレイで問題を自動的に「修正」することができます。

Prettierは素晴らしいプロジェクトだと確信していますが、個人的にはそれを一歩後退させたものと考えています。 そのコントロールを私から奪う。 TSLintはコードを「修正」する必要はありません。それが問題である場合は、問題としてフラグを立てるだけです。 この問題が複雑であることは間違いありませんが、eslintはそれを解決しました。 以前は機能していたルール。 それを壊すために何が変わったのですか？

@jscharettご

私はまた、Prettierが意見を述べていることに同意します。 私はむしろPrettierについてそれが好きです。 今、私のチームは、誰のフォーマットの意見がより合理的であるかについて議論する必要はありません。 私たちは皆、Prettierについて文句を言うことができます：笑い：。

編集：

以前は機能していたルール。 それを壊すために何が変わったのですか？

冒頭のコメントは、このルールが意図したとおりに機能していなかったと私に信じさせます。

Prettierは素晴らしいプロジェクトだと確信していますが、個人的にはそれを一歩後退させたものと考えています。 そのコントロールを私から奪う。

私の経験では、Prettierを追加するまでは、フォーマットルールをきめ細かく制御できると思っていました...そして、フォーマットの特定の方法についてはあまり気にしないことに気付いた直後に、それらは一貫してフォーマットされていました。 これ以上心配する必要がなく、コードの外観ではなく、コードに実行させたいことに完全に集中することは、大きな認知的負荷です。

tslintは、フォーマットのカテゴリに分類される他のものをすでに検証しています。 たとえば、変数名と演算子の間に配置、ブラケットスタイル、またはスペースを適用します。 さらに、よりきれいなような意見のある解決策に頼ることなく、インデントを検証できることが望ましい。

議論を減らし、PRを増やす必要があります。 😉😉

どうぞ@ffxsam

私のコメントは主に冗談でした。 なぜこの問題は1年以上前のものであり、進展が見られないのか疑問に思っていますが。 誰もがリンティングとプリティエについて議論しているようです。

@ffxsamtslintがts部分に関するものなのかlint部分に関するものなのかについて議論があるため

それは有効なポイントです。 TSLint / ESLintと重複しているようです。 しかし、機能していないindentオプションがあり、その期間を知っている人のために壊れているという事実は残っています。 TSLintコードベースに精通している人がそれを修正するのが最も速く/最も簡単なことのようです...？

x spaces => y spaces修正に投票してください。 これは、当社が非常に大きく依存している機能です。 これを修正しないだけでは意味がありません。

@ffxsam私はこの問題をほぼ1年間見ています、ええ、それは長いです、しかしあなたが見ることができるように2つのPRの試みがあり、そしてそれは今までうまくいきませんでした、私はそれが見た目より難しいと思います、しかしメンテナにとってはもちろん簡単かもしれませんが、私にはたくさんの忍耐力があります：slightly_smiling_face：

空のプロジェクトでも再現可能
https://github.com/dimaShin/tslint-reproduce-2814

こんにちは@dimaShin 、再現可能なリポジトリを作成するために時間を割いていただきありがとう

おそらく修正オプションを使用して、この機能を待っていますが、それは私にとっては問題ではありません。 前回チェックしたとき、人々はインデントとtslintをチェックするためにprettierを使用していました。

私はそれがあなたにぴったりだと言っているわけではありません、私にとっては確かにそうではありません、私はまたこの特定のオプションに.editorconfigを使用し、これが解決されたら後でtslintに切り替えることをお勧めします。

繰り返しになりますが、情報を追加するために時間を割いていただきありがとうございます:)

インデントをチェックするための戦略を決定しましょう。 参考までに、eslintが使用する戦略は次のとおりです。

1. OffsetStorageインスタンスは、必要なオフセットのマップを格納します。各トークンには、別の指定されたトークンまたは最初の列からの指定されたオフセットがあります。
2. ASTがトラバースされるときに、それに応じてトークンの必要なオフセットを変更します。 たとえば、BlockStatementを入力するときは、BlockStatement内のすべてのトークンをBlockStatementの最初の中括弧から1インデントレベルだけオフセットします。
3. ASTをトラバースした後、OffsetStorageコンテナーに従って、すべてのトークンの予想されるインデントレベルを計算します。
4. 各行について、最初のトークンの予想されるインデントをファイル内の実際のインデントと比較し、2つの値が等しくない場合はトークンを報告します。

この戦略は構文に基づいていますが、以前のコメントに基づくと、特定の方法で行を分割する必要があるため、あまりにも意見が分かれています。 コードがどのように行に分割されるかに関係なく、インデントをチェックする簡単な方法を提案します。

1. インデントの単位は設定で定義されています。 タブの場合、これは1つのタブ文字です。 スペースの場合、2文字または4文字のスペース文字です。
2. ファイルの最初の空でない行には、インデントの単位がゼロである必要があります
3. 後続のすべての空でない行には、次のいずれかが必要です。
   NS。 前の空でない行と同じ数のインデントまたは
   NS。 前の空でない行としてのインデントの単位数未満または
   NS。 前の空でない行のインデントの単位数よりも正確に1つ多い

議論するいくつかの追加事項：

• サイズパラメータはタブには影響しません（なぜですか？）
• サイズパラメータを正の整数にすることができない理由はありません
• インデントサイズの自動修正は、構文戦略ルートを使用しないと実装できない可能性があります

@stifflerusまた、ルールは{}ブロックのないif / for / arrow関数で機能する必要があります

@maximelkinあなたが与えた例は2行目にインデントされていませんか？ 提案された戦略が失敗する場所の例を挙げていただけますか？

if(this) that(); //okay because it's all one line

if(this)
  that(); //also okay because the second line is indented

let x = () => f(); //okay because it's all one line

let y = () =>
  f(); // I have not seen any code but like this but it would be okay

これが修正されたら素晴らしいでしょう。

インデントルールの適用がまだ機能していないので、基本的なことは信じられません。

それで、2018年にそのようなtsコードをリントする方法はありませんか？

const x = {
  a: 1,
   b: 2,
}

私のために働く
./.eslintrc.ts.js ：

module.exports = {
  'parser': 'typescript-eslint-parser',
  'parserOptions': {
    'ecmaVersion': 6,
    'sourceType': 'module',
    'ecmaFeatures': {
      'jsx': true,
    }
  },
  'plugins': [
    'react',
  ],
  'rules': {
    'indent': ['error', 2],
  },
}

yarn eslint --no-eslintrc --config ./.eslintrc.ts.js --ext .tsx src

https://github.com/eslint/typescript-eslint-parser

角度のインデント問題について私が見つけた解決策は、次のステップでよりきれいに追加することです。
npm install --save-dev tslint-plugin-prettier prettier tslint-jasmine-rules
tslint.jsonを編集->
`" rulesDirectory "：[
"node_modules / codelyzer"、
"node_modules / tslint-plugin-prettier"、
"node_modules / tslint-jasmine-rules / dist"

]、
"extends"： "tslint-plugin-prettier"、

「ルール」：{
「よりきれい」：本当、
//ここに必要なルールを追加します `

そしてpackege.jsonに追加->
"、"よりきれい "：{
「singleQuote」：true、
"printWidth"：140、
「半」：本当、
「bracketSpacing」：true、
"arrowParens"： "常に"、
"パーサー"： "typescript"
}」

これは、ほとんどの人が日常的に使用している機能です。 この問題がもっと注目されれば幸いです。

みんな、tslint-eslint-rulesが提供するter-indentオプションを使用できます。

"ter-indent"：[true、4、{"SwitchCase"：1}]

それは私のために働いた。 乾杯！

@hiteshaleriyaは私がしばらくの間私のプロジェクトで持っていたものですが、実際にはエラーを修正しておらず、ただそれらを黙らせています...これが私のtslint.jsonです：

{
    "extends": "tslint-config-airbnb",
    "rules": {
        "ter-indent": ["error", "spaces", 4],
        "no-unused-vars": ["warn"],
        "no-multi-spaces": false,
        "no-console": false,
        "max-line-length": false,
        "import-name": false
    }
}

警告やエラーを発生させずに終了した関連する例を次に示します。

function retrieveAndSetConfig(): Promise<any> {
  return new Promise((resolve, _) => {
  // ^ 2 spaces, expected 4
    const ghe = new GHEUtils();
    // ^ 4 spaces, expected 8
    // ...
}

また、タブが使用されている場合もエラーは表示されません（ただし、4つのスペースが存在する場合は仕様による可能性がありますか？）。

@SpencerKaiser以下に示すように、ter-indentルールを更新してから、次のことを試してください。

"ter-indent": [true, 4]

私はあなたの例を試しました、そしてそれは私の終わりに期待通りに働いています（エラーを引き起こします）。

@hiteshaleriyaすぐに返事をくれてありがとう！ そのため、期待どおりにエラーがスローされるようになりました（👍）が、 --fixエラーを修正していません。 何か案は？

@ SpencerKaiser--fixコマンドを2回実行してみてください。 最初は最初の行だけをインデントし、2回目は残りをインデントします（サンプルコードの場合）。 奇妙に思えますが、機能しない場合は問題を報告してください。

@hiteshaleriyaなので、いくつかの観察結果...もう一度実行する必要はありませんn/4約nはスペースのインデントの長さです。プロジェクトで最もインデントされた行¯\ _（ツ）_ /¯

最終的にすべてを終えた後、次のような基本的なインデントエラーをスキップしているようです。

class Something {
    function myFunc() {
        const myThing = {
            wat: 1,
         wattt: 5,    // 9 spaces, expected 12
        };
    }
}

const （17行目）のインデントレベルを0スペースに変更すると、 --fix省略したときに、スペースのある行を除外して、残りのエラーのフラグが立てられます。

ERROR: 17:1  ter-indent  Expected indentation of 8 spaces but found 0.
ERROR: 18:1  ter-indent  Expected indentation of 4 spaces but found 12.
ERROR: 20:1  ter-indent  Expected indentation of 0 spaces but found 8.

--fixの場合、最初のパスは次のとおりです。

        const myThing = {
    wat: 1,
         wattt: 5,
};

そして2番目のパス：

        const myThing = {
            wat: 1,
         wattt: 5,
        };

考え??

tslint-config-prettierでその問題をどのように解決したか

@shubich私は同じことをすることになった...

これに関する更新はありますか？

@MaKCbIMKo私が理解している限り、チーム全体がeslint visit typescript-eslintの統合に移行し、近い将来tslintは非推奨になるため、今のところこのルールを無視する（またはtslint-config-prettierを使用する）のと同じくらい良いでしょう。 ）

このタスクの複雑さとプロジェクトの方向性の変更による終了：＃4534

typescript-eslintのESLintルールは私にとって完璧に機能します（＃4534を参照）：

module.exports = {
    "env": {
        "browser": true,
    },
    "parser": "@typescript-eslint/parser",
    "parserOptions": {
        "ecmaVersion": 2019,
        "sourceType": "module",
        "ecmaFeatures": {
            "jsx": true
        },
        "project": "./tsconfig.json",
    },
    "plugins": ["@typescript-eslint"],
    "rules": {
        "@typescript-eslint/indent": ["error", 2] // or ["error", "tab"]
    }
}

tslint-config-prettierでその問題をどのように解決したか

リンクが見つかりません。

🤖ビープ音！ 👉TSLintは非推奨です👈そしてtypescript-eslintに切り替える必要があります！ 🤖

🔒この問題は、これ以上の不必要な議論を防ぐためにロックされています。 ありがとうございました！ 👋

PS tslint-config-prettier -TypeScriptを_フォーマット_するために、TSLintなどの_linters_の使用を停止してください。 これは、 Prettierなどの_formatter_を使用する方が適切です。