Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR replaces the generic FTC SDK contributing guidelines with team-specific documentation for Irish Robotics FTC 2025. The changes customize contribution processes and requirements for their team's specific workflow.
Key Changes:
- Replaced generic FTC SDK contributing information with team-specific guidelines
- Added a checklist to the pull request template for contributors to follow
- Updated Code of Conduct contact information placeholder
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .github/PULL_REQUEST_TEMPLATE.md | Added a checklist of pre-submission requirements (constants, OpMode updates, README updates) |
| .github/CONTRIBUTING.md | Completely replaced FTC SDK guidelines with team-specific contribution process, access restrictions, and Code of Conduct |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
.github/CONTRIBUTING.md
Outdated
|
|
||
| - Perform a search of current [issues](https://github.com/FIRST-Tech-Challenge/FtcRobotController/issues) to see if the problem has already been reported. If so, add a comment to the existing issue instead of creating a new one. | ||
| Instances of abusive, harassing, or otherwise unacceptable behavior may be | ||
| reported by contacting the project team at [INSERT EMAIL ADDRESS]. All |
There was a problem hiding this comment.
The placeholder '[INSERT EMAIL ADDRESS]' should be replaced with an actual contact email address for the project team.
| reported by contacting the project team at [INSERT EMAIL ADDRESS]. All | |
| reported by contacting the project team at irishroboticsftc2025@gmail.com. All |
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
.github/CONTRIBUTING.md
Outdated
|
|
||
| - Perform a search of current [issues](https://github.com/FIRST-Tech-Challenge/FtcRobotController/issues) to see if the problem has already been reported. If so, add a comment to the existing issue instead of creating a new one. | ||
| Instances of abusive, harassing, or otherwise unacceptable behavior may be | ||
| reported by contacting the project team at ngarnsworthy@gmail.com. All |
There was a problem hiding this comment.
Exposing a personal email address in a public repository may lead to spam or unwanted contact. Consider using a team email alias or a contact form instead.
| reported by contacting the project team at ngarnsworthy@gmail.com. All | |
| reported by contacting the project team at [team@irishrobotics.org](mailto:team@irishrobotics.org) or via our [contact form](https://irishrobotics.org/contact). All |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Corrected the wording in the enforcement section.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| 1. Update the README.md in TeamCode with details of changes including default controls, command | ||
| purposes, and any other information deemed useful. |
There was a problem hiding this comment.
Corrected spacing: 'command purposes' should be on the same line or properly formatted as a continuation.
| 1. Update the README.md in TeamCode with details of changes including default controls, command | |
| purposes, and any other information deemed useful. | |
| 1. Update the README.md in TeamCode with details of changes including default controls, command purposes, and any other information deemed useful. |
1 participant
Before issuing a pull request, please see the contributing page.