Hello and thank you for contributing to Team 501 the PowerKnights' robot program! Below are some rules regarding contributing to this repo. Please read this before making a PR and especially before pushing master.
If you are a member of the team you have full push access to the repo. This means that you can push to the repo without having to fork it and make a PR from that fork to the master branch. So the following process should take place when you want to make changes to the program:
- Make an issue
- Add a clear description of what you're trying to solve
- Assign to yourself or another member
- Add tags that align with the goal of the issue
- Make a branch with a name that clearly defines what feature you are trying to add/which bug you are trying to fix
- Commit your changes to that branch
- Make your PR
- Set @Cal-Hagner and @Java4First as a reviewer
- Close issue from PR with the following in your description:
This PR resolves {NUMBER OF ISSUE GOES HERE}
- Once all the checks have passed, wait for one of the reviewers to approve and then you're clear to merge your branch! If you are merging, please remember to squash and merge to keep commit history clean. Likely, one of the two moderators will have already squash-merged your PR.
- Delete the branch that you created for the PR
Same process for a team member except you need to fork the repo and then make your PR from that Fork. You also don't have access to putting tags or reviewers to your PR.
Thank you again!