-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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 start commands to devcontainer #3902
feat: add start commands to devcontainer #3902
Conversation
I did not add docs because I don't want to confuse people with these commands, but maybe you have an idea where to add a hint that these commands exist? |
To not change existing behaviour, I did place |
I think you need to fix the ci builds. I think it's ok to left the docs blank for this pr. |
Description
I work with the dify repo every week and every time I forget the whole command to start the api or the worker. I added these commands because they are intuitive and easy to remember.
This change just introduces shortcut commands to run services, which are made available via alias in the devcontainer:
Type of Change
How Has This Been Tested?
~/.bashrc
you should see 4 commands defined at the end of the file~/.bashrc
againSuggested Checklist:
dev/reformat
(backend) andcd web && npx lint-staged
(frontend) to appease the lint godsoptional
I have made corresponding changes to the documentationoptional
I have added tests that prove my fix is effective or that my feature worksoptional
New and existing unit tests pass locally with my changes