You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Protected files were modified in this change.
This pull request is in request_review mode and requires explicit human scrutiny before merge.
Protected files: .github/workflows/ci.yml
🤖 This is an automated PR from Repo Assist.
What changed: The Linux CI job's "Install build SDK and integration test runtime" step hardcoded 9.0.309 when invoking dotnet-install.sh, duplicating the version already declared in global.json. This PR reads the SDK version from global.json via jq instead, so future SDK bumps only need to touch one file.
Why:global.json is the single source of truth for the SDK version used by setup-dotnet@v4 earlier in the same job. Keeping a second hardcoded copy risks drift (e.g. someone bumps global.json but forgets the CI script line, or vice versa) — a low-risk maintenance/engineering cleanup.
Trade-offs: Adds a dependency on jq being present on the ubuntu-latest runner, which it is by default (GitHub-hosted runners ship jq preinstalled).
Test Status
This is a CI-workflow-only change (no F#/build changes). Not run through dotnet test/build.sh since no source code was touched. Validated locally:
jq -r '.sdk.version' global.json correctly extracts 9.0.309 from the current global.json.
YAML structure otherwise unchanged (single shell-script line edited).
The change will be exercised for real on the next CI run of this PR, which will confirm the extracted version matches and the SDK install step still succeeds.
As always, human review before merge is appreciated — I never merge PRs myself.
Generated by 🌈 Repo Assist, see workflow run. Learn more. Comment /repo-assist to run again
Add this agentic workflow to your repo
To install this agentic workflow, run
gh aw add githubnext/agentics/workflows/repo-assist.md@ae8d551f07c7ed7619f8c58c7bb4c3ac89395d38
Note
This was originally intended as a pull request, but the git push operation failed.
Original error: The process '/usr/bin/git' failed with exit code 1
Caution
Protected files were modified in this change.
This pull request is in
request_reviewmode and requires explicit human scrutiny before merge.Protected files:
.github/workflows/ci.yml🤖 This is an automated PR from Repo Assist.
What changed: The Linux CI job's "Install build SDK and integration test runtime" step hardcoded
9.0.309when invokingdotnet-install.sh, duplicating the version already declared inglobal.json. This PR reads the SDK version fromglobal.jsonviajqinstead, so future SDK bumps only need to touch one file.Why:
global.jsonis the single source of truth for the SDK version used bysetup-dotnet@v4earlier in the same job. Keeping a second hardcoded copy risks drift (e.g. someone bumpsglobal.jsonbut forgets the CI script line, or vice versa) — a low-risk maintenance/engineering cleanup.Trade-offs: Adds a dependency on
jqbeing present on theubuntu-latestrunner, which it is by default (GitHub-hosted runners shipjqpreinstalled).Test Status
This is a CI-workflow-only change (no F#/build changes). Not run through
dotnet test/build.shsince no source code was touched. Validated locally:jq -r '.sdk.version' global.jsoncorrectly extracts9.0.309from the currentglobal.json.The change will be exercised for real on the next CI run of this PR, which will confirm the extracted version matches and the SDK install step still succeeds.
As always, human review before merge is appreciated — I never merge PRs myself.
Add this agentic workflow to your repo
To install this agentic workflow, run
Note
This was originally intended as a pull request, but the git push operation failed.
Original error: The process '/usr/bin/git' failed with exit code 1
Workflow Run: View run details and download bundle artifact
The bundle file is available in the
agentartifact in the workflow run linked above.Create the pull request manually