-
-
Notifications
You must be signed in to change notification settings - Fork 184
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Introduce Modelina CLI that can be extended by AsyncAPI CLI #1724
Comments
@jonaslagoni I am happy to work on this issue and the idea is very useful to encourage reusability. |
Hey @ankur0904 any progress or update? 🙂 |
@jonaslagoni My sincere apologies for the delayed response. Unfortunately, I faced unexpected challenges that prevented me from making progress on the issue. |
Hi @ankur0904, are you still working on this one? If not, @jonaslagoni I would like to work on this. |
@devilkiller-ag Please go ahead as I am currently not working on this issue. |
Thanks @ankur0904! @jonaslagoni I will study a little bit about this issue and let you know my initial findings. |
I think we can close this issue, as the remaining effort is already under way in asyncapi/cli#1376 |
Reason/Context
Currently, you can use the AsyncAPI CLI to generate models with Modelina, however, one limitation is that only AsyncAPI inputs are supported.
A better setup would be to have the AsyncAPI CLI extend the Modelina CLI which has no restrictions on inputs. That way we can reuse most if not all the implementation while keeping everything Modelina-related in this repo.
Here is the TODO I would suggest to follow to solve this issue:
The text was updated successfully, but these errors were encountered: