-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
cli: add lb4 relation command #2223
Conversation
@TomerSalton Thank you for the contribution. Please sign CLA by clicking https://cla.strongloop.com/agreements/strongloop/loopback-next. |
To fix the coding style/format issues, run |
solution for issue #1359:loopback-next support lb4 relation command adding cli relation command
solution for issue #1359:loopback-next support lb4 relation command adding new files for lb4 cli relation command
solution for issue #1359:loopback-next support lb4 relation command after npm run lint:fix
we are testing the commit procedure of CLI relation new command #2223
@@ -0,0 +1,9 @@ | |||
<component name="InspectionProjectProfileManager"> |
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.
Please remove all .idea
file from the commit. In general, I am not a fan of listing different IDEs in our .gitignore
, but considering the popularity of JetBrain IDEs, it may be a sensible thing to do.
@raymondfeng @strongloop/loopback-maintainers what's your opinion? We are keeping VS Code settings inside our git repo, should we perhaps promote JetBrains IDEs like IDEA and WebStorm as supported IDEs and version their configuration files too?
See also https://github.com/strongloop/loopback-next/blob/master/docs/site/WEBSTORM.md
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.
I tend to keep IDE settings inside our project, it would be very friendly for beginners to build/test/debug.
adding fk input for user prompt fix #1359
Closing in favor of #2426 |
add lb4 relation command
Solution for issue #1359: loopback-next supports lb4 relation command.
Checklist
npm test
passes on your machinepackages/cli
were updatedexamples/*
were updated