Closed Biki-das closed 1 year ago
We have these basic policies to make the approval process smoother for our volunteer team.
Please make sure your code passes all tests and there are no merge conflicts.
The process helps maintain accurate and well-formatted documentation and is a prerequisite for getting your PR approved. Assigned reviewers regularly review the PR queue and tend to focus on PRs that are passing.
When your PR has been assigned reviewers contact them to get your code reviewed and approved via:
Your reviewer(s) will have the following roles:
Read our CONTRIBUTING.md file. Most importantly:
@Biki-das Thank you! Its going to take me a few days to give this the time it needs.
@Biki-das Thank you! Its going to take me a few days to give this the time it needs.
sure :-)
@Cioppolo14 thanks for the review , let me fix the sections as you suggested!
@Cioppolo14 the recent commit only includes the wording and sentences changes, the sidebar changes as you requested would be done in another commit or else it might became hard for you to go through :-)
@Cioppolo14 done with the changes, and tried my best to articulate the changes as you requested! looking forward to your review!
Sidebar has been revamped
@Cioppolo14 changes pushed! thanks for the feedback! we really need a linter as i had to manually lint so many files. this becomes too cumbersome!
@Cioppolo14 thanks for the cooperation and feedback , without this, it won't have been possible to get so far with this one. Looking forward to contributing more!
What kind of change does this PR introduce?
feature
Issue Number:
Fixes #624
Did you add tests for your changes?
N/A
Snapshots/Videos:
If relevant, did you update the documentation?
N/A
Summary
This PR adds a Git guide for contributors, it has git and code setup guide along with rules related to creating Pull request , commits etc Does this PR introduce a breaking change?
NO
Other information
Have you read the contributing guide? Yes