-
Notifications
You must be signed in to change notification settings - Fork 139
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
Add C9 template #215
Add C9 template #215
Conversation
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.
Thank you, @rezabekf ! I have left comments and suggestions.
Co-authored-by: Matteo Rinaudo <52245107+mrinaudo-aws@users.noreply.github.com>
Thanks Chris and Matteo, i have applied all your suggestions and requested round two of the review. Cheers! |
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.
Looking good, some minor corrections/broken links and suggestions to make it clearer between Cloud9 and Local Dev
Co-authored-by: Chris Harding <112917432+cjhar@users.noreply.github.com>
Hi @mrinaudo-aws, all the requested changes has been applied. Please review. |
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.
Hi @rezabekf ! Made minor suggestions - please review; otherwise LGTM.
Co-authored-by: Matteo Rinaudo <52245107+mrinaudo-aws@users.noreply.github.com>
Co-authored-by: Matteo Rinaudo <52245107+mrinaudo-aws@users.noreply.github.com>
Co-authored-by: Matteo Rinaudo <52245107+mrinaudo-aws@users.noreply.github.com>
Co-authored-by: Matteo Rinaudo <52245107+mrinaudo-aws@users.noreply.github.com>
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
What does this PR do and why?
This PR revamps the Pre-requisities of the workshop as we are adding Cloud( as recommended way to do the labs.
Issue #, if available
PR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
make version part=<major|minor|patch>
?By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.