-
Notifications
You must be signed in to change notification settings - Fork 108
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
Blog: Hacktoberfest 2022 #1320
Blog: Hacktoberfest 2022 #1320
Conversation
3c14dfa
to
2f64985
Compare
Looks good! |
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 just one minor item.
2f64985
to
34f6a94
Compare
Probably good to post. |
34f6a94
to
441684f
Compare
FYI changed to WIP as it should not be merged until tomorrow 😉 |
Now I feel I could have changed the date :). But anyways :). |
We've published the September Update blog post today so wouldnt have wanted another one until tomorrow at the earliest 😉 |
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.
Looks good but added a few comments. Please feel free to ignore the suggestions if you feel otherwise :).
d943431
to
f4f084b
Compare
Thank you! |
Looks good thanks! @ricekot do you want to review again? |
Signed-off-by: ArkaprabhaChakraborty <[email protected]>
f4f084b
to
5b91947
Compare
Pushed the changes should be good now |
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.
Thanks!
Signed-off-by: ArkaprabhaChakraborty [email protected]