Add `convert` command about cli HOT 8 CLOSED

Hi all! 👋 @magicmatatjahu @Souvikns @derberg
I've been having a look and investigating a bit on this issue and I'd like to start working on it as a starting point to contribute to the CLI repo. If no objections from your side, please let me work on it and I'll get back to you with a proposal as soon as I have some working code 💪 Cheers!

from cli.

Maybe we have a --output flag that takes a path as an input. Converts the old version to the new version and outputs in that path, and if the --output flag is missing we can just update the original file.

from cli.

Our comverter-js has builded custom CLI, but we should move it to the main CLI.

definitely, same with generator. This is the ultimate goal of this CLI. One CLI to rule them all 😄

I would definitely like CLI to be friendly and interactive, and when you do asyncapi convert, you get an interactive response from the CLI with list of versions that you can convert too, so you select the version that you want manually. And of course you can skip this interactiveness with a flag like --targetVersion or -t.

the future of the command
my opinion is that we should not put = between asyncapi convert and the JavaScript Converter that we have. Meaning: that now asyncapi convert will invoke the list of available target versions, but simply because in code we will by default support AsyncAPI now, but in future this should be definitely extendable, so we can add other formats, why not 🤷🏼

--output
imho we should do --output as it will be universal for other commands, not >. And if you do not provide --output or -o then we print the result in the terminal

from cli.

@peter-rr go ahead mate!

from cli.

This issue has been automatically marked as stale because it has not had recent activity 😴
It will be closed in 60 days if no further activity occurs. To unstale this issue, add a comment with detailed explanation.
Thank you for your contributions ❤️

from cli.

Solution that we should consider for that issue asyncapi/community#249 It's only an idea so please treat it as suggestion, not final approach. Feel free to comment :)

from cli.

I think this issue can be closed as it was implemented by @peter-rr in #188. @Souvikns @magicmatatjahu can you close it? I can do it but I should not 😄

from cli.

Yeah, it's implemented. Thanks!

from cli.