-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Arooj #3192
Arooj #3192
Conversation
🙏 Thanks for your pull request @arooj-12, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated! Some of the most popular are PR Statistics
|
Please only add one file with your name/submission @arooj-12 |
🙏 Thanks for your pull request @arooj-12, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated! Some of the most popular are PR Statistics
|
@arooj-12 are you still interested in this PR? |
Yes I am. I have only pushed one file.
…On Sun, 5 May 2024, 8:22 pm Sophia Brandt, ***@***.***> wrote:
@arooj-12 <https://github.com/arooj-12> are you still interested in this
PR?
—
Reply to this email directly, view it on GitHub
<#3192 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A62XFIFOVHMBYC5VSAAR7MLZAZFB5AVCNFSM6AAAAABGHHFVHSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJUHA2DSMRQGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@arooj-12 As you can see in the files tab on this page there are changes to 3 files. Please fix that, otherwise we cannot accept your pull request. |
Hi @arooj-12, I think you are getting confused with the requested changes, actually the reviewer is asking you to do the following changes:
NOTES
|
@arooj-12 Hey, just letting you know that I'm going to close your PR in a few days. |
No description provided.