solidjs / solidjs/solid-vite-plugin
I updated `vite-plugin-solid` and now namespaces merged with components are no longer allowed
Nessuno ha ancora preso questa issue.
- Lingua principale
- TypeScript
- Stelle
- 520
- Fork
- 70
- Merge medio
- 23h 35m
- PR unite (30g)
- 39
Descrizione
Describe the bug
If I create a .tsx file with this content
function A() {
return <>1</>
}
namespace A {
export const a = 1;
}
console.log(A);
It throws this error when I load the page (Running vite dev)
Transform failed with 1 error:
ERROR: The symbol "A" has already been declared
I tried downgrading vite-plugin-solid and it worked
npm i vite-plugin-solid@2.8.0
(On the playground it always works)
Your Example Website or App
(localhost)
Steps to Reproduce the Bug or Issue
- Put a merged namespace declaration WITH A COMPONENT somewhere
- Run
vite dev - Go to the web page
Expected behavior
I expected this allowed TS syntax to not stop working
Screenshots or Videos
No response
Platform
- OS: Windows
- Browser: Edge
- Version: 120.0.2210.77
Additional context
No response
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia con l'esempio .tsx fornito, che contiene un componente e un namespace unito, quindi esegui vite dev per riprodurre l'errore di trasformazione. Confronta il comportamento con vite-plugin-solid@2.8.0; l'attività è completata quando la sintassi TypeScript viene caricata senza l'errore di simbolo duplicato nella versione corrente del plugin.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript, vite
- Ambito
- build-system
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100