LuaLS / LuaLS/lua-language-server
feature request: refactor to expand and collapse expressions
まだ誰も着手していません。
- 主要言語
- Lua
- スター
- 4.4k
- フォーク
- 442
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
It would be nice to be able to pull out repeated expressions or expand an expression on demand.
So one could jump back and forth between for example:
function love.draw()
local w, h = love.graphics.getDimensions()
love.graphics.setColor(1, 0, 0)
love.graphics.circle('fill', 50, 50, 22)
love.graphics.setColor(1, 1, 0)
love.graphics.circle('fill', 50, 100, 22)
love.graphics.setColor(0, 1, 0)
love.graphics.circle('fill', 50, 150, 22)
end
and
local g = love.graphics
function love.draw()
local w, h = g.getDimensions()
g.setColor(1, 0, 0)
g.circle('fill', 50, 50, 22)
g.setColor(1, 1, 0)
g.circle('fill', 50, 100, 22)
g.setColor(0, 1, 0)
g.circle('fill', 50, 150, 22)
end
It would also be nice to do this for local variables, so that the change is only applied where all the nessecary variables are visible (i.e. for required modules within a scope or long paths in arguments), so one can easily switch between
local function compileInfo(project)
return {
root = project.main,
compiler = project.conf.toolchain == 'gnu' and 'gcc'
or project.conf.toolchain == 'windows' and 'msvc'
or project.conf.toolchain == 'clang' and 'clang'
}
end
and
local function compileInfo(project)
local c = project.conf
return {
root = project.main,
compiler = c.toolchain == 'gnu' and 'gcc'
or c.toolchain == 'windows' and 'msvc'
or c.toolchain == 'clang' and 'clang'
}
end
or
local function compileInfo(project)
local t = project.conf.toolchain
return {
root = project.main,
compiler = t == 'gnu' and 'gcc'
or t == 'windows' and 'msvc'
or t == 'clang' and 'clang'
}
end
This is most usefull for long namespaced paths, but it may also be usefull to extract out some repeated constants or function calls and expand them again, so one can switch between:
local function copy(src, dst)
for f in lfs.dir(src) do
local src = io.open(src..package.config:sub(1, 1)..f, "rb")
local dst = io.open(dst..package.config:sub(1, 1)..f, "wb")
end
end
and
local DIR_SEP = package.config:sub(1, 1)
local function copy(src, dst)
for f in lfs.dir(src) do
local src = io.open(src..DIR_SEP..f, "rb")
local dst = io.open(dst..DIR_SEP..f, "wb")
end
end
Ideally one could chose the name of the pulled out expression via an prompt.
It's a long time since I had a good look at your codebase, so I wouldn't know where to start if I wanted to implement such a feature myself. I might be willing to try, if someone gives me a few pointers though.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
この issue にはファイル、テスト、エントリポイントが記載されていません。まず language server のリファクタリングまたは code-action の実装を把握し、スコープ解析、名前の入力要求、展開、取り消し可能な変換をどのように仕様化してテストするかを判断してください。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- lua
- 領域
- devtools
- issue の種類
- 機能追加
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 25/100