Skip to content

net 10 upgrade complete#317

Merged
StuartFerguson merged 1 commit intomainfrom
task/#307_net_10_upgrade
Dec 9, 2025
Merged

net 10 upgrade complete#317
StuartFerguson merged 1 commit intomainfrom
task/#307_net_10_upgrade

Conversation

@StuartFerguson
Copy link
Member

closes #307
closes #313
closes #309

Comment on lines 106 to 136

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {}

Copilot Autofix

AI about 2 months ago

To fix this, add an explicit permissions: block near the top level of the workflow file, just after the name: (or right after on:), to restrict the GITHUB_TOKEN permissions. The safest default is permissions: {} (no permissions), but often some minimal permissions are needed, e.g., contents: read for workflows that need read access to the codebase. If jobs require other permissions (such as for uploading releases or working with issues), add only those required permissions. Since this workflow appears to only build, upload artifacts, and (optionally) interact with NuGet feeds (which doesn't use the GITHUB_TOKEN), setting contents: read as the default is safe and minimal. If finer control is needed, set permissions at the job level. The file to update is .github/workflows/createrelease.yml, and the change is at the very top of the file, after the name: line.


Suggested changeset 1
.github/workflows/createrelease.yml

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/.github/workflows/createrelease.yml b/.github/workflows/createrelease.yml
--- a/.github/workflows/createrelease.yml
+++ b/.github/workflows/createrelease.yml
@@ -1,4 +1,6 @@
 name: Release
+permissions:
+  contents: read
 
 on:
   release:
EOF
@@ -1,4 +1,6 @@
name: Release
permissions:
contents: read

on:
release:
Copilot is powered by AI and may make mistakes. Always verify output.
@StuartFerguson StuartFerguson merged commit de7225f into main Dec 9, 2025
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update Shared Nugets Move to Windows Release Workflow Net 10 Upgrade

1 participant