-
-
Notifications
You must be signed in to change notification settings - Fork 228
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: Improved the Current CLI. #556
Conversation
✅ Deploy Preview for robyn canceled.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also Fixes #560 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Mr-Sunglasses 👋
Great work! 🔥 I have some minor suggestions. We can merge it after that! ✨
CodSpeed Performance ReportMerging #556 will not alter performanceComparing Summary
|
@sansyrox I've made the changes in this PR s requested. Do tell me if any more changes to be made or if not it is ready to merge 🌮 . |
026cc08
to
a8cf7f0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Mr-Sunglasses 👋
Thank you for addressing the suggestions. Just two more suggestions before we can merge it 😄
71f36b6
to
416bc97
Compare
@sansyrox 🎉 I have removed the --version flag and just realized 🤔 there's no need for it! One can check if Robyn is installed just by typing python3 -m robyn --help. 🚀 I think this PR is now ready to be merged! 💯 |
feat: Improved the current --create flag of robyn cli with new style Update the requirement.txt for new deps in cli fix: Removed some mistaken additon of --version flag in docs [fix]: Updated the link of robyn docs fixed the doc's link Removed --version docs. removed --version flag from CLI removed --version flag from CLI
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Great work @Mr-Sunglasses 🔥
Description
feat: Improved the current --create flag of robyn cli with new style
feat: added the new --version flag in robyn cli to check the currently installed version of robyn
Screen.Recording.2023-07-17.at.4.56.04.AM.mov
This PR fixes #511 (Note - This PR fixes a small portion of issue #511, there is a complete roadmap of features in the issue which need to be added to completely fix this issue )