Skip to content

Conversation

@jmeridth
Copy link
Contributor

@jmeridth jmeridth commented Jul 16, 2025

Pull Request

First PR to see if this works. If it does, will add PRs to the other OSPO GitHub Actions.

Proposed Changes

This ensures that when we have the copilot agent writing PRs, it sets up its environment correctly and understands our standards for coding.

  • Copilot Instructions file (docs)
  • Copilot Setup Steps (docs)

Readiness Checklist

Author/Contributor

  • If documentation is needed for this change, has that been included in this pull request
  • run make lint and fix any issues that you have introduced
  • run make test and ensure you have test coverage for the lines you are introducing
  • If publishing new data to the public (scorecards, security scan results, code quality results, live dashboards, etc.), please request review from @jeffrey-luszcz

Reviewer

  • Label as either fix, documentation, enhancement, infrastructure, maintenance or breaking

Copilot AI review requested due to automatic review settings July 16, 2025 14:40
@jmeridth jmeridth requested a review from a team as a code owner July 16, 2025 14:40
Copy link
Contributor

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 adds GitHub Copilot agent configuration files to enable proper environment setup and establish coding standards for the project. The changes focus on integrating Copilot coding agent support with automated dependency installation and clear development guidelines.

  • Adds automated workflow for Copilot environment setup with Python and dependency installation
  • Establishes coding standards and development flow instructions for Copilot agent

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/copilot-setup-steps.yml Workflow configuration for Copilot agent environment setup with Python and dependency installation
.github/copilot-instructions.md Development guidelines and coding standards for Copilot agent interactions

@jmeridth
Copy link
Contributor Author

This will hopefully prevent us from having to request copilot from running make lint (example)

Copy link
Member

@zkoppert zkoppert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Will keep an eye out for what else we can add as we go along. 🚀

Signed-off-by: Zack Koppert <[email protected]>
jmeridth and others added 3 commits July 16, 2025 13:12
Co-authored-by: Zack Koppert <[email protected]>
Signed-off-by: Jason Meridth <[email protected]>
and fix warning about incorrect pylint disable name

change `too-many-positional-arguments` to `too-many-arguments`

Signed-off-by: jmeridth <[email protected]>
one must upgrade pylint before running make lint. 🤦

Signed-off-by: jmeridth <[email protected]>
@jmeridth jmeridth merged commit 4229a2d into main Jul 17, 2025
31 checks passed
@jmeridth jmeridth deleted the jm_copilot_setup branch July 17, 2025 15:00
jmeridth added a commit to github/issue-metrics that referenced this pull request Aug 20, 2025
jmeridth added a commit to github/issue-metrics that referenced this pull request Aug 20, 2025
jmeridth added a commit to github/contributors that referenced this pull request Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants