Skip to content

Fix Copilot setup workflow tool ordering #1032

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.

Sign up for GitHub

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

Merged
BenjaminMichaelis merged 5 commits into from
Apr 26, 2026
Merged

Fix Copilot setup workflow tool ordering #1032

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3eede13
fix: install copilot tooling before build validation
Copilot Apr 26, 2026
c3abfb0
fix: harden copilot setup tool installation
Copilot Apr 26, 2026
4b05af6
fix: avoid noisy dotnet-ef fallback output
Copilot Apr 26, 2026
008fa6b
fix: simplify dotnet-ef presence check
Copilot Apr 26, 2026
f9f2713
fix: pin bash shell for setup tool script
Copilot Apr 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 18 additions & 16 deletions .github/workflows/copilot-setup-steps.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,11 @@ jobs:
env:
NUGET_AUTH_TOKEN: ${{ secrets.AZURE_DEVOPS_PAT }}

- name: Set up Node.js for frontend development
uses: actions/setup-node@v6
with:
node-version: '24'

- name: Set up dependency caching for faster builds
uses: actions/cache@v5
id: nuget-cache
Expand All @@ -40,6 +45,19 @@ jobs:
${{ runner.os }}-nuget-${{ hashFiles('**/packages.lock.json') }}
${{ runner.os }}-nuget-

- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v4

- name: Install additional development tools
shell: bash
run: |
echo "Installing additional tools for Copilot agent environment..."
if dotnet tool list --global | grep -q 'dotnet-ef'; then
dotnet tool update --global dotnet-ef
else
dotnet tool install --global dotnet-ef
fi
Comment on lines +54 to +59
Copy link

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

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

The workflow already pins dotnet-ef via the local tool manifest (.config/dotnet-tools.json), but this step installs/updates a global dotnet-ef to whatever the latest version is at runtime. That can introduce non-deterministic builds and version mismatches vs the repo’s pinned EF tooling (e.g., 10.0.7). Prefer running dotnet tool restore and using the restored tool (or install/update the global tool to the manifest’s pinned version) instead of updating to latest.

Suggested change
echo "Installing additional tools for Copilot agent environment..."
if dotnet tool list --global | grep -q 'dotnet-ef'; then
dotnet tool update --global dotnet-ef
else
dotnet tool install --global dotnet-ef
fi
echo "Restoring local .NET tools for Copilot agent environment..."
dotnet tool restore

Copilot uses AI. Check for mistakes.

- name: Restore with dotnet
run: dotnet restore

Expand All @@ -48,19 +66,3 @@ jobs:

- name: Run .NET Tests
run: dotnet test --no-build --configuration Release

- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v4

- name: Set up Node.js for frontend development
uses: actions/setup-node@v6
with:
node-version: '24'

- name: Install additional development tools
run: |
# Install common development tools that Copilot agents might need
echo "Installing additional tools for Copilot agent environment..."

# Install EF Core tools globally
dotnet tool install --global dotnet-ef
Loading