-
Notifications
You must be signed in to change notification settings - Fork 190
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
docs: PR Template in CONTRIBUTING.md is corrected #241
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.
@shruti1421 Although, the changes looks fine, still why the link is not working from your branch on your forked repo.
@Rahulm2310 Now, you can check I've added the correct link in docs/CONTRIBUTING.md file too. |
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.
Works fine
Working!. @shruti1421 just a suggestion. Both of your commits have the same commit message. Please be more specific regarding the commit messages like what this commit is fixing or changing. For now, thanks for your contribution 🎉 |
@keshakaneria As this PR already has 2 reviews, it is ready to merge. |
We followed this in OSH program last time if it has 2 reviews, then for final check by the maintainers to merge them so I added the same. Correct me if I'm wrong😄 |
Apologies for the labels @Rahulm2310, I cleared about this with OS Leaders, thus changing the labels accordingly 🚀 |
@shruti1421 can you resolve conflicts please |
I will close this PR due to lack of activity. Thank you @shruti1421 for contributing 🤗 |
Description
Now, if we click "this template" in CONTRIBUTING.md file PR template page opens successfully.
Fixes #230
Type of Change:
Delete irrelevant options.
Code/Quality Assurance Only
How Has This Been Tested?
If you are working on a design then add a gif to show the responsiveness of your design.
Please follow testing guidelines and share the link of deployed site here.
Checklist:
Delete irrelevant options.
Code/Quality Assurance Only