Integrate `asyncapi/diff` in the CLI about cli

[derberg]

So I guess, we will have
asyncapi diff <file1> <file2l>
for file.

yup, but with clear indication which one is checked against the other, I mean if file1 is checked against file 2? or the opposite

the part about file flag I mentioned (that it will be removed) is something that @Souvikns will work on #37 (comment)

[Souvikns]

Yeah sure, let me know if you need anything.

[Souvikns]

hmm, it doesn't.

Lukasz said that this functionality should be a separate implementation independent of the command.

Yeah, you should create a wrapper around diff and connect it with CLI. For a little context, the CLI is this, so you just need to create a function wrapping your diff library and create a new command for it.

Can you give me some context on what kind of output diff is generating? As of now, all output is printed in the console itself.

[github-actions]

Welcome to AsyncAPI. Thanks a lot for reporting your first issue. Please check out our contributors guide and the instructions about a basic recommended setup useful for opening a pull request.

Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

[derberg]

Hey, there is a topics covered here that are important no matter if we work on diff or not: the output

atm, library outputs on terminal only. There is no --output option so far. This is independent from diff as we need output to file functionality in many different commands, for relations finding or for optimizations. This needs to be implemented separately

regarding optional flags, we so far tend to always have name, so instead of --json do --format=json

as if it comes to asyncapi diff --files <file1.yml> <file2.yml>, we have some discussions in other issue that are kind of going the direction of getting rid of --file or --files flag, as flags are options, they are optional, while in case of file, it is must have, not optional, it is kinda a center of all the things

[aayushmau5]

atm, library outputs on terminal only. There is no --output option so far. This is independent from diff as we need output to file functionality in many different commands, for relations finding or for optimizations. This needs to be implemented separately

Got it.

regarding optional flags, we so far tend to always have name, so instead of --json do --format=json

👍

as if it comes to asyncapi diff --files <file1.yml> <file2.yml>, we have some discussions in other issue that are kind of going the direction of getting rid of --file or --files flag, as flags are options, they are optional, while in case of file, it is must have, not optional, it is kinda a center of all the things

So I guess, we will have

asyncapi diff <file1> <file2l>

for file.

[aayushmau5]

Now that the migration to Oclif is done, I will start working on this. Hope to make a PR by the end of this week.

[aayushmau5]

@Souvikns Does the CLI have the ability to output data into files? Since, Lukasz said that this functionality should be a separate implementation independent of the command.

[aayushmau5]

Can you give me some context on what kind of output diff is generating? As of now, all output is printed in the console itself.

Right now I'm just generating the default json output.

[magicmatatjahu]

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 :)

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this issue, add a comment with a detailed explanation.

There can be many reasons why some specific issue has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this issue forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

[aayushmau5]

Should we close this issue now that we have integrated diff in CLI?