Skip to content

Commit 2f849a5

Browse files
richlanderCopilot
andauthored
[infra] Fix jq example in update-os-coverage skill (#126572)
> [!NOTE] > This PR was AI/Copilot-generated. This updates the `update-os-coverage` skill guidance so the example tag check fails explicitly when the requested prereqs image tag is missing. - replace the `jq -r ... | .[]` example with `jq -e ... any(...)` - make the missing-tag case unambiguous and easier to script Addresses skipped review feedback from #126524 (discussion_r3034880248). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent f68ea32 commit 2f849a5

File tree

1 file changed

+1
-1
lines changed
  • .github/skills/update-os-coverage

1 file changed

+1
-1
lines changed

.github/skills/update-os-coverage/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ Before making any changes, confirm the **exact target container tag** exists in
9494
```bash
9595
TARGET_TAG="<exact-image-tag>"
9696
curl -sL https://github.com/dotnet/versions/raw/refs/heads/main/build-info/docker/image-info.dotnet-dotnet-buildtools-prereqs-docker-main.json \
97-
| jq -r --arg tag "$TARGET_TAG" '[.repos[].images[].platforms[].simpleTags[]] | unique | map(select(. == $tag)) | .[]'
97+
| jq -e --arg tag "$TARGET_TAG" 'any(.repos[].images[].platforms[].simpleTags[]; . == $tag)'
9898
```
9999

100100
If the exact tag is **not found in `image-info`**, stop and inform the user. Treat that as authoritative even if a registry lookup appears to work. The image must be created first at [dotnet/dotnet-buildtools-prereqs-docker](https://github.com/dotnet/dotnet-buildtools-prereqs-docker). Check if an open issue or PR already exists, for example:

0 commit comments

Comments
 (0)