koinos / koinos-proto

Official home of the Koinos Protobuf schema. Koinos data types are defined here and generated in a variety of supported languages.
MIT License
3 stars 7 forks source link

[BUG]: use npm package "@romainroe/as-proto-gen" instead of "as-proto-gen" in ci #114

Closed roaminro closed 2 years ago

roaminro commented 2 years ago

Is there an existing issue for this?

Current behavior

The official "as-proto-gen" package doesn't generate the correct AS output files so I had to fork the repo to fix it. I cannot merge the fork to the original repo as it introduces breaking changes and changes some of the logic of the original library.

The fork of the npm package is available at "@romainroe/as-proto-gen" The source code of this fork is hosted at https://github.com/roaminroe/as-proto/tree/0.3.0/packages/as-proto-gen This example repo uses the correct npm package: https://github.com/roaminroe/koinos-proto-as

Expected behavior

Generate the correct AS files (no namespaces and valid import paths)

https://github.com/koinos/koinos-proto/blob/as/ci/install.sh#L32 yarn add --dev as-proto-gen@$AS_PROTO_VERSION should be yarn add --dev @romainroe/as-proto-gen@$AS_PROTO_VERSION

Steps to reproduce

No response

Environment

- OS: all

Anything else?

No response