Skip to content

Conversation

@RyanCavanaugh
Copy link
Member

Copilot AI review requested due to automatic review settings June 24, 2025 16:21
@github-project-automation github-project-automation bot moved this to Not started in PR Backlog Jun 24, 2025
@typescript-bot typescript-bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Jun 24, 2025
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 introduces a new GitHub Actions workflow to configure and run Copilot setup steps on demand.

  • Adds a workflow_dispatch job named copilot-setup-steps with minimal permissions
  • Defines environment setup for Node.js, Rust, and Go, and runs dependency/install commands
  • Includes a formatting check step using the hereby tool
Comments suppressed due to low confidence (3)

.github/workflows/copilot-setup-steps.yml:28

  • The comment references 'dprint' caches but the step runs 'hereby'. Consider updating the comment to accurately describe pulling 'hereby' caches.
      # pull dprint caches before network access is blocked

.github/workflows/copilot-setup-steps.yml:29

  • Instead of appending || true to ignore failures, consider using the continue-on-error: true attribute on the step for clarity and consistency.
      - run: npx hereby check:format || true

.github/workflows/copilot-setup-steps.yml:18

  • [nitpick] Consider using the version tag @v4.2.2 instead of the full commit SHA to improve readability, or consistently pin all actions to specific SHAs for reproducibility.
      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2

@github-project-automation github-project-automation bot moved this from Not started to Needs merge in PR Backlog Jun 24, 2025
@jakebailey jakebailey merged commit 3af37ef into microsoft:main Jun 24, 2025
32 checks passed
@github-project-automation github-project-automation bot moved this from Needs merge to Done in PR Backlog Jun 24, 2025
@RyanCavanaugh RyanCavanaugh deleted the addCopilotInstructions branch June 24, 2025 17:00
@microsoft microsoft locked as resolved and limited conversation to collaborators Jan 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants