LuaLS / LuaLS/vscode-lua

Fix Indent affecting lines incorrectly

オープン
#169 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
TypeScript
スター
215
フォーク
62
PR マージ指標
30日以内にマージされた PR はありません

説明

Which OS are you using?

Windows

Expected Behaviour

Writing local ret = Button:new(text, x, y) and then putting in a new line before text, to change the arguments to a vertical list, un-indents at text. That is, the result is

Button:new(
text, x, y)
Actual Behaviour

The result should be

Button:new(
    text, x, y)
Reproduction steps
  • Write a function with an argument
  • Break before the argument
Additional Notes

The behaviour is as expected if Fix Indent is off. However I believe the point is for other cases like strings, to get

"I went to a function
the other day"

and not

"I went to a function
    the other day"

I'm not exactly sure what the point of the option is. I assume it doesn't apply to every time vscode indents code, but I think the set it's supposed to apply to probably shouldn't apply to the function(arg) case.

Log

No response

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず、Fix Indent を有効にした VSCode Lua extension で報告された動作を再現します。Lua 関数呼び出しの引数の前で改行し、複数行の文字列の場合と比較します。インデント処理のエントリーポイントを追跡し、両方のケースのカバレッジを追加します。引数はインデントされたまま、文字列の継続部分は誤ってインデントされない状態になれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
lua, typescript
領域
tooling
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。