-
-
Notifications
You must be signed in to change notification settings - Fork 765
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
feat: hacktoberfest Blog Post #1131
Conversation
✅ Deploy Preview for asyncapi-website ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-1131--asyncapi-website.netlify.app/ |
pages/blog/hacktoberfest-2022.md
Outdated
<table> | ||
<tr> | ||
<td align="center"> | ||
<a href="https://github.com/Amishakumari544"> | ||
<img src="https://avatars.githubusercontent.com/u/49230384?v=4" width="100px;" alt=""/> | ||
<br /> | ||
<sub> | ||
<b> | ||
Amishakumari544 - 1 PR | ||
</b> | ||
</sub> | ||
</a> | ||
</td> |
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.
I don't wanna make review for md content (please wait for @alequetzalli), but maybe will be better to create React component for that, hmm, wdyt? 🤔
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.
Maybe it should come in a separate PR?
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.
separate PR is the biggest lie of a person on open source 😄 I'll fix it right away but in separate PR
and PR will never be created XD That's only mu suggestion, you can create or not if you don't want. I don't have a problem with it.
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.
I suggested this because I also have a post on the conference where I'll be giving a special shoutout to the volunteers and I'd eventually be listing them like the one above... So the PR will eventually be created 😄
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.
OK, so PR will be (eventually) created, nice 😄
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.
Sorry, I'm not a md
reviewer 😅 , but I found some bugs inside the blog. Kindly go through it.
pages/blog/hacktoberfest-2022.md
Outdated
@@ -0,0 +1,207 @@ | |||
--- | |||
title: "Is Hacktoberfest Good For Maintainers?" |
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.
Is this the correct title for this PR? 🤔 Because in the blog post, you are summarising Hacktoberfest 2022 in AsyncAPI.
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.
yup, i think it is title of old existing article
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.
I copied from the old one with updating in mind and I forgot 😄
pages/blog/hacktoberfest-2022.md
Outdated
@@ -0,0 +1,207 @@ | |||
--- | |||
title: "Is Hacktoberfest Good For Maintainers?" | |||
date: 2020-11-05T06:00:00+01:00 |
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.
The date written is incorrect. Kindly change it.
pages/blog/hacktoberfest-2022.md
Outdated
|
||
<table> | ||
<tr> | ||
<td align="center"> |
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.
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.
yo, why not just make a normal markdown table? O_O it's a good practice (unlike this current mess haha) and you don't need to build a new component 😄
👉🏽👉🏽 here's a simple tool to generate your markdown tables: https://www.tablesgenerator.com/markdown_tables
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.
I feel using a component gives us more flexibility in terms of looks. Btw, I already created a PR
pages/blog/hacktoberfest-2022.md
Outdated
<tr> | ||
<td align="center"> | ||
<a href="https://github.com/Amishakumari544"> | ||
<img src="https://avatars.githubusercontent.com/u/49230384?v=4" width="100px;" alt=""/> |
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.
<img src="https://avatars.githubusercontent.com/u/49230384?v=4" width="100px;" alt=""/> | |
<img src="https://avatars.githubusercontent.com/Amishakumari544" width="100px;" alt=""/> |
I think directly using Github username in the URL to show the profile photos is a better option.
Thanks so much for writing this up 🙏🏼 So feedback:
Maybe mention all the contributors in this article, so they can verify on their own. I mean specify how many hacktoberfest contributions they did |
@Amishakumari544 @toukirkhan @manavdesai27 @Ankitchaudharyy @nawed2611 @dhfherna @amit-ksh @mastdev @codingtenshi @Mayaleeeee Please kindly confirm if the numbers of your PRs during hacktoberfest is right |
Yes mine is 2. |
I had 3 PR merged in hacktoberfest. |
yes 🙌 |
Yes, 1 PR merged during hacktoberfest |
|
Yeahh |
Co-authored-by: Alejandra Quetzalli <alejandra.quetzalli@postman.com>
Co-authored-by: Alejandra Quetzalli <alejandra.quetzalli@postman.com>
Co-authored-by: Alejandra Quetzalli <alejandra.quetzalli@postman.com>
Co-authored-by: Alejandra Quetzalli <alejandra.quetzalli@postman.com>
Co-authored-by: Alejandra Quetzalli <alejandra.olvera.novack@gmail.com>
Co-authored-by: Lukasz Gornicki <lpgornicki@gmail.com> Co-authored-by: akshatnema <20bcs022@iiitdmj.ac.in>
Co-authored-by: Akshat Nema <76521428+akshatnema@users.noreply.github.com>
Co-authored-by: Akshat Nema <76521428+akshatnema@users.noreply.github.com>
Updated the PR cc @alequetzalli @derberg |
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.
❤️
Co-authored-by: Lukasz Gornicki <lpgornicki@gmail.com>
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.
we merge it tomorrow
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.
approved 👍
/rtm |
/rtm |
I wrote a post summarising our participants at this year's Hacktoberfest
cc @derberg @alequetzalli