bytecodealliance / bytecodealliance/componentize-go

Proposed changes to `bindings --pkg-name`

Offen
#65 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Rust
Sterne
17
Forks
8
Ø Merge
6 Std. 57 Min.
Gemergte PRs (30 T.)
5

Beschreibung

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?

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, das Verhalten von `bindings --pkg-name` in `wit-bindgen-go` und `componentize-go` zu überprüfen, wobei die aktuelle Befehlssequenz im Issue als Ausgangspunkt dient. Die Arbeit ist abgeschlossen, wenn `componentize-go bindings --output internal --pkg-name internal` sowohl die Ordner `imports` als auch `exports` erstellt und die separate Option `--export-pkg-name` entfernt werden kann.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
go, wasm
Bereich
cli, tooling
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
48/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.