Vara's week 3 blog

[lb-97]

No description provided.

[shilpiprd]

All seems good!

[codecov]

Codecov Report

Merging #2828 (4aeadef) into master (3e9caf8) will increase coverage by 0.01%.
Report is 23 commits behind head on master.
The diff coverage is n/a.

❗ Current head 4aeadef differs from pull request most recent head be066a0. Consider uploading reports for the commit be066a0 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2828      +/-   ##
==========================================
+ Coverage   81.47%   81.48%   +0.01%     
==========================================
  Files         144      144              
  Lines       20054    20055       +1     
  Branches     3192     3192              
==========================================
+ Hits        16339    16342       +3     
  Misses       2906     2906              
+ Partials      809      807       -2     

see 3 files with indirect coverage changes

[skoudoro]

Hi @lb-97,

Overall, it looks good. See below few comments.

Do not hesitate to split in paragraph. it will make you text more readable.

[skoudoro]

Hi @lb-97, What is the update concerning this ?

[skoudoro]

Thank you for the update @lb-97 . Can you do split in multiple block like you did in the blog week4 ? thank you in advance

[lb-97]

Thank you for the update @lb-97 . Can you do split in multiple block like you did in the blog week4 ? thank you in advance

Done!

[skoudoro]

Thanks for the update @lb-97, merging