Skip to content

Conversation

@danmoseley
Copy link
Member

Description

Copilot currently can't build when assigned issues, because its firewall won't allow it to download anything. We could open the firewall, but recommendation is instead to create a yml that sets up the environment so it is all set for copilot to run with.

Following instructions from https://docs.github.com/en/copilot/customizing-copilot/customizing-the-development-environment-for-copilot-coding-agent#preinstalling-tools-or-dependencies-in-copilots-environment

@radical or @RussKie is this sufficient to build the repo such that nothing else needs downloading from that point on?

cc @captainsafia

Copilot AI review requested due to automatic review settings May 20, 2025 00:33
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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@github-actions github-actions bot added the area-engineering-systems infrastructure helix infra engineering repo stuff label May 20, 2025
@radical
Copy link
Member

radical commented May 20, 2025

The build step should be good. But there might be more dependencies when running tests, like Playwright which gets downloaded when preparing for helix, or running the tests. It might be useful to also pass /p:PrepareForHelix=true.

@danmoseley
Copy link
Member Author

added prepareforhelix.

@danmoseley danmoseley requested a review from radical May 20, 2025 00:52
@danmoseley danmoseley merged commit 1e8ee55 into dotnet:main May 20, 2025
251 of 252 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jun 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-engineering-systems infrastructure helix infra engineering repo stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants