Skip to content
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

feat: add Modelina CLI #1849

Merged
merged 7 commits into from
Mar 27, 2024
Merged

feat: add Modelina CLI #1849

merged 7 commits into from
Mar 27, 2024

Conversation

devilkiller-ag
Copy link
Member

@devilkiller-ag devilkiller-ag commented Mar 2, 2024

Description

Modelina and integrate it with AsyncAPI CLI

Related Issue

Related to issue #1724 and PR #1785

Checklist

  • The code follows the project's coding standards and is properly linted (npm run lint).
  • Tests have been added or updated to cover the changes.
  • Documentation has been updated to reflect the changes.
  • All tests pass successfully locally.(npm run test).

Additional Notes

This PR is the extension of PR #1785. The initial base has been built from Async API CLI. Now we need to add rectify all the tests, do clean-ups, and integrate this Modelina CLI with Modelina CLI.

Copy link

netlify bot commented Mar 2, 2024

Deploy Preview for modelina canceled.

Name Link
🔨 Latest commit c2814e1
🔍 Latest deploy log https://app.netlify.com/sites/modelina/deploys/65f9c9ca940a9f00088e80f1

@devilkiller-ag devilkiller-ag changed the title feat: create initial modelina cli (#1785) feat: develop Modelina CLI Mar 2, 2024
@devilkiller-ag devilkiller-ag self-assigned this Mar 2, 2024
@devilkiller-ag
Copy link
Member Author

cc: @jonaslagoni

@jonaslagoni
Copy link
Member

Perfect @devilkiller-ag 👌

@jonaslagoni jonaslagoni changed the title feat: develop Modelina CLI feat: add Modelina CLI Mar 2, 2024
@devilkiller-ag
Copy link
Member Author

@jonaslagoni, so what should be our next step?

@jonaslagoni
Copy link
Member

@devilkiller-ag we each can now make PRs that target the cli branch, and we just collaborate ✌️ You continue exactly as you have been doing, lets just stick to small changes so we dont duplicate work as much as possible 🙂How does that sound?

@devilkiller-ag
Copy link
Member Author

Okay, then I will start with working on tests, and see why it's failing?

@jonaslagoni
Copy link
Member

@devilkiller-ag alright 👍

@devilkiller-ag
Copy link
Member Author

Just merged the cleanup, what's the next thing to tackle 👀

Copy link

sonarcloud bot commented Mar 19, 2024

Quality Gate Passed Quality Gate passed

Issues
29 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
2.7% Duplication on New Code

See analysis details on SonarCloud

@jonaslagoni jonaslagoni changed the base branch from master to next March 22, 2024 19:51
@jonaslagoni
Copy link
Member

Retarget the PR to next so we can release it ASAP and do a bit of trial running making sure everything works before it enters master.

@jonaslagoni
Copy link
Member

@devilkiller-ag I actually see it as merge ready 🤔 How about you?

@devilkiller-ag
Copy link
Member Author

devilkiller-ag commented Mar 22, 2024

@jonaslagoni Seems good to me! But the Runtime test for Rust, Kotlin, and Scala Model is still failing. Is it fine for merging to next with these issues?

@jonaslagoni
Copy link
Member

@devilkiller-ag they can safely be ignored, has nothing to do with these changes

@devilkiller-ag
Copy link
Member Author

Hi @jonaslagoni, when we are going to merge this?

@jonaslagoni
Copy link
Member

Wanted to wait few days to give the other maintainers some time to review it, but since it's next lets see how it behaves ✌️

@jonaslagoni jonaslagoni merged commit ff28234 into next Mar 27, 2024
37 of 40 checks passed
@jonaslagoni jonaslagoni deleted the cli branch March 27, 2024 19:19
@asyncapi-bot
Copy link
Contributor

🎉 This PR is included in version 4.0.0-next.21 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants