-
Notifications
You must be signed in to change notification settings - Fork 8
Cloud&ai dev #210
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
Cloud&ai dev #210
Conversation
adamblanchard
left a comment
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.
Overall this is totally great for the first PR. I left a few comments with questions and requested changes which should be quick.
There is only one structural difference that i pointed out, regarding the module weeks. But, it's fine to merge this PR and tackle that later, in my opinion.
If you can get back to my questions and my requested changes, i'm super happy to approve it. Big thanks for getting this into the program repo!
|
Hey @najibkado, this is looking great. We can get this merged asap, and yes feel free to keep adding content as you run the course for the first time :-) The only thing to fix before we can merge is the linting is failing. If you run ./lint you can see the errors and fix them locally. "./lint --fix" can fix some things automatically. If you need any help, just ping me. |
Hi,
Here's is a first draft of the Cloud & AI Course, Can you review the format and the content?