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

Update README.md by adding links issue #1198 #1222

Closed
wants to merge 1 commit into from

Conversation

aditi5050
Copy link

solved issue Improvement in the README.md #1198

@aditi5050
Copy link
Author

@tinniaru3005 can you please have a look

@tinniaru3005
Copy link
Member

@tinniaru3005 can you please have a look

Hey @aditi5050, Congrats on making your first PR to the project. :)
The changes look fine to me. But it would be great if you follow a few things while making PR

  • If you want to solve an issue, ask for it in the comments
  • Mostly, the one who asks first gets assigned
  • If the issue is assigned to you, then only start solving it
  • After solving, send a PR

The issue which you are linking has already been solved and a PR is made to solve it. There are a lot of issues on all the repositories. Try looking for them. If you think you can solve an issue which is currently not assigned to anyone then, ask for it in the comments. Do tag me so that I get a notification and can assign the issue to you. :)
If you have more doubts feel free to ask it on Zulip.

@aditi5050
Copy link
Author

@tinniaru3005 thanks a lot for guiding me it will surly me in the future and it could be a great help if would assign me an issue I'll work on it :)

@tinniaru3005
Copy link
Member

@tinniaru3005 thanks a lot for guiding me it will surly me in the future and it could be a great help if would assign me an issue I'll work on it :)

Ya sure. Just look at the issues which are not assigned to anyone. Ask for it in the comment by tagging me(so that I get a notification). I will assign it to you. Also, you should be the first to ask. I hope that helps :)

@aditi5050 aditi5050 closed this Oct 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants