leoafarias/fvm

[BUG] `fvm use` makes unrelated changes to settings.json.

Open

#635 aberto em 20 de fev. de 2024

Ver no GitHub
 (4 comments) (5 reactions) (1 assignee)Dart (277 forks)batch import
buggood first issuehelp wanted

Métricas do repositório

Stars
 (5.419 stars)
Métricas de merge de PR
 (Mesclagem média 1d 11h) (12 fundiu PRs em 30d)

Description

Before creating a bug report please make check the following

  • You have read our FAQ
  • [n/a] If you have used flutter. Please install correctly, run pub cache repair. Close the terminal and try again.
  • [n/a] If you are on Windows. Make sure you are running the terminal as administrator or with developer permissions.
  • [n/a] Run fvm doctor if possible and add the output to the issue.

Describe the bug When running fvm use, the settings.json file is update to set the new version to use. However, comments are removed from the file and tabs are changed to spaces.

To Reproduce Steps to reproduce the behavior:

  1. Add comments to settings.json.
  2. Set it to save tabs, not spaces.
  3. Run fvm use stable
  4. Comments will be removed and tabs will be changed to spaces.

Expected behavior Only the version number should be changed. Tab/space indentation should match rest of file.

Guia do colaborador

Direção de pesquisa
Modifique o código que grava settings.json para preservar a formatação existente, como comentários e indentação.
Pilha de tecnologia
dart
Domain
cli
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
DartGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75