-
Notifications
You must be signed in to change notification settings - Fork 0
Add CONTRIBUTING info #20
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
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.
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 |
Copilot
AI
Oct 23, 2025
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.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 |
Copilot
AI
Oct 23, 2025
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.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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. |
Copilot
AI
Oct 23, 2025
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.
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. |
Before issuing a pull request, please see the contributing page.