-
Notifications
You must be signed in to change notification settings - Fork 23
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
docs: add the new build tools and adjust supported technologies #539
Conversation
Signed-off-by: behnazh-w <[email protected]>
* npm | ||
* Yarn | ||
* Go | ||
* Docker container |
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.
* Docker container | |
* Docker |
Because I think Docker is a tool while Docker container in this case is not.
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.
See commit 4afc607
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. The PR can be merged once the two questions are addressed.
Please approve again based on the new changes. |
Signed-off-by: behnazh-w <[email protected]>
No description provided.