Skip to content
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

added blog posts for GSoC'21 #477

Merged
merged 4 commits into from Jul 30, 2021
Merged

Conversation

antrikshmisri
Copy link
Member

This PR adds my blog posts for Google Summer of Code 2021.

@codecov
Copy link

codecov bot commented Jul 27, 2021

Codecov Report

Merging #477 (ffa1dd2) into master (f376819) will increase coverage by 0.11%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #477      +/-   ##
==========================================
+ Coverage   88.41%   88.53%   +0.11%     
==========================================
  Files          31       31              
  Lines        6519     6575      +56     
  Branches      782      787       +5     
==========================================
+ Hits         5764     5821      +57     
  Misses        535      535              
+ Partials      220      219       -1     
Impacted Files Coverage Δ
fury/ui/tests/test_helpers.py 100.00% <0.00%> (ø)
fury/ui/helpers.py 98.21% <0.00%> (+5.35%) ⬆️

Copy link
Contributor

@skoudoro skoudoro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @antrikshmisri,

Thank you for this PR. See below some comments. Please, try to render the docs locally to see if everything is ok.

Thank you @SunTzunami for the first review !

Can you please also review this @devmessias. Thanks!

docs/source/posts/2021/2021-06-08-week-1-antriksh.rst Outdated Show resolved Hide resolved
docs/source/posts/2021/2021-06-08-week-1-antriksh.rst Outdated Show resolved Hide resolved
docs/source/posts/2021/2021-06-13-week-2-antriksh.rst Outdated Show resolved Hide resolved
docs/source/posts/2021/2021-06-21-week-3-antriksh.rst Outdated Show resolved Hide resolved
docs/source/posts/2021/2021-07-05-week-5-antriksh.rst Outdated Show resolved Hide resolved
docs/source/posts/2021/2021-07-12-week-6-antriksh.rst Outdated Show resolved Hide resolved
docs/source/posts/2021/2021-07-12-week-6-antriksh.rst Outdated Show resolved Hide resolved
docs/source/posts/2021/2021-07-19-week-7-antriksh.rst Outdated Show resolved Hide resolved
docs/source/posts/2021/2021-07-26-week-8-antriksh.rst Outdated Show resolved Hide resolved
@skoudoro skoudoro requested a review from devmessias July 28, 2021 10:59
@skoudoro skoudoro added this to the v0.8.0 milestone Jul 28, 2021
@antrikshmisri
Copy link
Member Author

Hey @skoudoro @SunTzunami , thanks for the review. All comments have been addressed. Thanks!

@skoudoro
Copy link
Contributor

@devmessias, we are waiting for your feedback before merging this PR. Please, take a look as soon as possible

@devmessias
Copy link
Contributor

devmessias commented Jul 29, 2021

Hi @skoudoro and @antrikshmisri . I had read it and looks very good to me. Maybe I miss something but my english is not good enough to find any typo.

@skoudoro
Copy link
Contributor

Thank you @antrikshmisri, merging!

@skoudoro skoudoro merged commit 9b52255 into fury-gl:master Jul 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants