PowerShell / PowerShell/vscode-powershell

RFC: Change Pipeline Indentation Style default from `NoIndentation` to `IncreaseIndentationForFirstPipeline`

Aperta
#4,296 3 commenti 8 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Area-Code Formatting Issue-Discussion
Lingua principale
TypeScript
Stelle
1.9k
Fork
548
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Prerequisites
  • I have written a descriptive issue title.
  • I have searched all issues to ensure it has not already been reported.
Summary

I've discovered that our default value to configure PSScriptAnalyzer's PipelineIndentation setting is NoIndentation, which differs from PSSA's own default (and arguably more popular) of IncreaseIndentationForFirstPipeline. This was last updated in May 2020 via https://github.com/PowerShell/vscode-powershell/pull/2698 which alludes to a bug in PSSA that I hope is since fixed.

@bergmeister in particular, what do you think?

Proposed Design

Just update the default in package.json to IncreaseIndentationForFirstPipeline. While I think this is a very desirable behavior, it would come at the cost of user-facing change that would affect formatting of scripts. So the question is, have the vast majority of people already overridden it with IncreaseIndentationForFirstPipeline?

Further RFC: Should we do a just an overhaul of all our default code formatting settings to match PSSA's defaults?

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia da package.json e confronta il valore predefinito attuale di PipelineIndentation con la documentazione di PipelineIndentation di PSScriptAnalyzer. Il lavoro mirato è completato quando il valore predefinito viene modificato in IncreaseIndentationForFirstPipeline; la questione più ampia dell’allineamento di tutti i valori predefiniti di formattazione richiede una decisione di un maintainer prima di ampliare l’ambito.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
typescript
Ambito
tooling
Tipo di issue
Funzionalità
Difficoltà
1/5
Tempo stimato
Meno di un'ora
Stato di attività
Tranquilla
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.