-
Notifications
You must be signed in to change notification settings - Fork 551
[copilot] setup .github/workflows/copilot-setup-steps.yml
#10193
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
Co-authored-by: jonathanpeppers <[email protected]>
Co-authored-by: jonathanpeppers <[email protected]>
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
{"version": "0.0.1", "workloads": { "android-deps": { "extends" : [ $(AndroidWorkloadExtendsElement) ] } } } | ||
{"version": "0.0.1", "workloads": { "android-deps": { "extends" : [ "microsoft-net-runtime-android" ] } } } |
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.
Previously, this was installing microsoft-net-runtime-android-aot
:
Installing pack Microsoft.NETCore.App.Runtime.AOT.linux-x64.Cross.android-x86 version 10.0.0-preview.6.25304.106...
Which failed on Github actions on linux.
I think we only need "microsoft-net-runtime-android"
here.
This comment was marked as outdated.
This comment was marked as outdated.
.github/workflows/copilot-setup-steps.yml
A successful GitHub action run here: |
@copilot check the changes again |
I've reviewed the current changes. The copilot-setup-steps.yml workflow has been successfully streamlined according to your feedback: ✅ Changed .NET version from 10.x to 9.x The workflow now only contains the essential steps: checkout, setup .NET 9.x, run jenkins build, and upload logs on failure. The ConfigureLocalWorkload.targets file correctly uses only "microsoft-net-runtime-android" for Linux to avoid the AOT installation failures. The GitCommitsInRange tasks have ContinueOnError="true" to handle git history issues gracefully. All requested changes have been implemented. (55fa5ff) |
Context: https://docs.github.com/en/copilot/customizing-copilot/customizing-the-development-environment-for-copilot-coding-agent
Setup
.github/workflows/copilot-setup-steps.yml
, which is a Github actionthat runs (only on Ubuntu) prior to Copilot doing its work.
This makes Copilot more useful, as it has a locally cloned and built source tree
with all dependencies installed. Copilot will be able to run various commands
to achieve tasks as assigned.
Other changes:
<GitCommitsInRange/>
Copilot must keep some custom
git
history, as this command was failing during our build:We can ignore the error, and just use 0 for the commit range if this occurs.
android-deps
workload for provisioning Mono, can simply use:Previously, it was failing to install a non-existent pack on Linux: