feat: Remove mutable props concept

Aperta
#5,644 1 commento 4 reazioni 1 assegnatario Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@rwaskiewicz ci sta già lavorando.

Dal 9/4/2024.

Valutazione

Questa issue non è ancora stata valutata.

Descrizione

triage
Prerequisites
Describe the Feature Request

As it stands now, prop being mutable doesn't do anything other than suppress a warning about modifying a prop.
Is there a reason that warning was put in?
This seems like a needless behavior that only worsens DX, and makes testing components harder.

Describe the Use Case
  • Do two-way sync of props with external store
  • Modify props in tests
  • Reset prop values on hot-module-replacement
Describe Preferred Solution

Remove the useless warning

Describe Alternatives

OR deprecate the concept of non-mutable props
OR add a stencil config option to make all props mutable

Related Code

No response

Additional Information

Related to https://github.com/ionic-team/stencil/issues/3490

Lingua principale
TypeScript
Stelle
13.2k
Fork
865
Merge medio
2g 16h
PR unite (30g)
29

Preparare l'ambiente

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.

Altre issue di stenciljs/core

Tutte le issue di stenciljs/core

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.