Skip to content

Conversation

@ngarnsworthy
Copy link
Collaborator

Before issuing a pull request, please see the contributing page.

Copilot AI review requested due to automatic review settings October 23, 2025 15:53
@ngarnsworthy ngarnsworthy linked an issue Oct 23, 2025 that may be closed by this pull request
Copy link

Copilot AI left a 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.


- 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
Copy link

Copilot AI Oct 23, 2025

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.

Suggested change
reported by contacting the project team at [INSERT EMAIL ADDRESS]. All
reported by contacting the project team at irishroboticsftc2025@gmail.com. All

Copilot uses AI. Check for mistakes.
@ngarnsworthy ngarnsworthy requested a review from Copilot October 23, 2025 15:57
Copy link

Copilot AI left a 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.


- 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
Copy link

Copilot AI Oct 23, 2025

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.

Suggested change
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

Copilot uses AI. Check for mistakes.
ngarnsworthy and others added 2 commits October 23, 2025 10:58
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Corrected the wording in the enforcement section.
@ngarnsworthy ngarnsworthy requested a review from Copilot October 23, 2025 15:59
Copy link

Copilot AI left a 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>
@ngarnsworthy ngarnsworthy requested a review from Copilot October 23, 2025 15:59
Copy link

Copilot AI left a 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>
@ngarnsworthy ngarnsworthy requested a review from Copilot October 23, 2025 16:00
Copy link

Copilot AI left a 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.

Comment on lines +11 to +12
1. Update the README.md in TeamCode with details of changes including default controls, command
purposes, and any other information deemed useful.
Copy link

Copilot AI Oct 23, 2025

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.

Suggested change
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.

Copilot uses AI. Check for mistakes.
@ngarnsworthy ngarnsworthy merged commit 9b33f8f into master Oct 23, 2025
1 check passed
@ngarnsworthy ngarnsworthy deleted the 18-update-all-contributing-files branch October 23, 2025 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update all contributing files

2 participants