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

GSoC: Final Report #845

Merged
merged 12 commits into from
Feb 23, 2024
Merged

GSoC: Final Report #845

merged 12 commits into from
Feb 23, 2024

Conversation

JoaoDell
Copy link
Contributor

Hello everyone, here is my final report. There are some things missing, like the 11° and 12° blogpost Fury links that are yet to be merged, but in general, I think that is it. I am gonna keep making some changes through the rest of this week until the deadline, but I created this PR so you could already give it a look
image

@codecov
Copy link

codecov bot commented Aug 24, 2023

Codecov Report

Merging #845 (b24a274) into master (e0741be) will not change coverage.
Report is 24 commits behind head on master.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #845   +/-   ##
=======================================
  Coverage   84.49%   84.49%           
=======================================
  Files          44       44           
  Lines       10454    10454           
  Branches     1411     1411           
=======================================
  Hits         8833     8833           
  Misses       1252     1252           
  Partials      369      369           

Copy link
Contributor

@ganimtron-10 ganimtron-10 left a comment

Choose a reason for hiding this comment

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

Hey @JoaoDell ,
I tried building this, everything worked fine instead of the weekly table at the bottom. I tried twice but I am not getting the table as you can see below.
image

Also I noticed the below comments.

@ganimtron-10
Copy link
Contributor

Hey @JoaoDell ,
Maybe you could use the format I used my final blogpost its much simpler - #847.
Before implementing, please confirm whether this table works or not.

@JoaoDell
Copy link
Contributor Author

@ganimtron-10 thanks for pointing that out, I just realised there was an extra | at line 12. I have erased it and now it is building fine 👍

@skoudoro skoudoro merged commit 52b3a9b into fury-gl:master Feb 23, 2024
22 of 30 checks passed
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.

3 participants