-
Notifications
You must be signed in to change notification settings - Fork 163
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
Week 5 blogpost #815
Week 5 blogpost #815
Conversation
Codecov Report
@@ Coverage Diff @@
## master #815 +/- ##
==========================================
- Coverage 84.43% 84.38% -0.05%
==========================================
Files 43 43
Lines 10274 10274
Branches 1393 1393
==========================================
- Hits 8675 8670 -5
- Misses 1239 1243 +4
- Partials 360 361 +1 |
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 @JoaoDell, the blogpost was generated successfully, and overall looks great. Please take a look at some minor comments I made.
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.
Hey @JoaoDell,
Nice Blogpost, LGTM.
Thanks for the review @tvcastillod and @ganimtron-10, just pushed the changes you pointed out 👍 |
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, merging
Hello, here is my week 5 blogpost: