asyncapi / bundler

Combine multiple AsyncAPI specification files into one.
Apache License 2.0
30 stars 16 forks source link

fix: tool file corrected #112

Closed akshatnema closed 1 year ago

akshatnema commented 1 year ago

Description The .asyncapi-tool file is corrected according to the errors made while validating with the schema.

aeworxet commented 1 year ago

This PR doesn't introduce a one property rename to simply align it's name with the existing schema like PR #109. Phrase

If your code generator generates Python code, then specify Python, even if code
generator itself is written in JavaScript.

raised several questions:

@derberg @magicmatatjahu

(due to holiday season, answers might delay up to after ~Jan 10, 2023)

akshatnema commented 1 year ago

If your code generator generates Python code, then specify Python, even if code generator itself is written in JavaScript.

@aeworxet The language parameter inside .asyncapi-tool file doesn't specify the code generated by the generator. Rather, it specifies the primary language with which the generator has been made. And it was decided before that we will have a single primary language for each tool, which is majorly being used to build the tool. Hence, it is a string, not an array.

aeworxet commented 1 year ago

Even if there was a decision made, but change was forgotten to be introduced into the schema, the https://github.com/asyncapi/website/blob/master/scripts/tools/tools-schema.json currently still reads in the description of the language property

If your code generator generates Python code, then specify Python, even if code
generator itself is written in JavaScript.

so I have to hit the brakes on this PR until senior engineers come to any agreement.

magicmatatjahu commented 1 year ago

@akshatnema @aeworxet language refers to the language that you can use in the "runtime". That is, for generators it should be the language that the generator generates, but bundler even though it operates on YAML and JSON it does not mean that these are the target language, it's JS/TS because in this language it's written. It's a information if you can use given lib/script/api in your codebase :)

aeworxet commented 1 year ago

Should the description https://github.com/asyncapi/website/blob/master/scripts/tools/tools-schema.json be altered then? Made more clear maybe?

aeworxet commented 1 year ago

/rtm

asyncapi-bot commented 1 year ago

:tada: This PR is included in version 0.3.10 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

magicmatatjahu commented 1 year ago

@akshatnema Next time please use chore: