bytecodealliance / bytecodealliance/componentize-go

Proposed changes to `bindings --pkg-name`

Aperta
#65 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Rust
Stelle
17
Fork
8
Merge medio
6h 57m
PR unite (30g)
5

Descrizione

I think that the `bindings --pkg-name` option should automatically create `imports` and `exports` folders. This would mean that we could also remove the `bindings --export-pkg-name` option. I think this would offer more clarity and less friction. This will require edits to both `wit-bindgen-go` and `componentize-go`.

Just in case the above isn't clear:

Current:
```
$ mkdir -p internal/exports
$ componentize-go \
bindings \
--output internal/imports \
--pkg-name internal/imports \
--export-pkg-name internal/exports
$ mv internal/imports/wit_exports imports/exports
$ ls internal
exports imports
```

Proposed:
```
$ componentize-go bindings --output internal --pkg-name internal
$ ls internal
exports imports

```

@dicej Any thoughts/concerns?

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia esaminando il comportamento di `bindings --pkg-name` in `wit-bindgen-go` e `componentize-go`, usando la sequenza di comandi attuale dell’issue come riferimento. Il lavoro è completato quando `componentize-go bindings --output internal --pkg-name internal` crea entrambe le cartelle `imports` e `exports` e l’opzione separata `--export-pkg-name` può essere rimossa.

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

Valutazione

Stack tecnologico
go, wasm
Ambito
cli, tooling
Tipo di issue
Funzionalità
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Tranquilla
Chiarezza
Abbastanza chiara
Idoneità per principianti
48/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.