-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Conversation
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.
It's great having you contribute to this project
Welcome to the community 🤓If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord chat and our GitHub Organisation - we help and encourage each other to contribute to open source, little and often 🤓 . Any questions let us know.
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.
[ Looks good 🧨. ]
Thanks @Pradumnasaraf , you have been a very good mentor for me in my open source journey. From finding how to start on twitter to doing my first pr and joining a community , you have always helped me. Thank you so much. |
hey @kabir0x23 thanks for reviewing the PR but i just had a question do i need to do anything else to get my PR merged since i dont know why it is still not merged?? BTW sorry for disturbing you. |
@Priyansh61 only a maintainer can merge your pr, so we will have to wait till a maintainer comes and merges it. Hope this helped. Thanks. |
Thank you, @Priyansh61 for your kind words and support. |
Hey!, we merge PR after 2 approval (From the maintainers), but I am merging this as this is not affecting any code of development. :) |
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.
Added Priyansh61 to README.md