Fix Indent affecting lines incorrectly
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- TypeScript
- Estrelas
- 215
- Forks
- 62
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Descrição
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
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece reproduzindo o comportamento relatado na VSCode Lua extension com o Fix Indent habilitado: faça uma quebra de linha antes de um argumento em uma chamada de função Lua e compare-a com uma string multilinha. Rastreie o ponto de entrada do tratamento de indentação e adicione cobertura para ambos os casos; considera-se concluído quando os argumentos permanecem indentados, enquanto a continuação da string não é indentada incorretamente.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- lua, typescript
- Domínio
- tooling
- Tipo de issue
- Bug
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 35/100