Skip to content

net 10 upgrade completed#660

Merged
StuartFerguson merged 1 commit intomainfrom
task/#651_net_10_upgrade
Dec 11, 2025
Merged

net 10 upgrade completed#660
StuartFerguson merged 1 commit intomainfrom
task/#651_net_10_upgrade

Conversation

@StuartFerguson
Copy link
Member

closes #651
closes #652
closes #654

Comment on lines 108 to 138

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 the problem, we need to add a permissions block to limit the GITHUB_TOKEN's scope. This can be done at the root of the workflow (to apply to all jobs), or per job (for fine-grained control). The best approach without functional changes is to insert a top-level permissions: block right after the workflow name: and before the on: key, setting the minimal necessary permissions.

From the workflow, it appears that the jobs mostly use actions such as artifact upload/download and possibly push to package feeds using secrets. By default, actions/upload-artifact and actions/download-artifact do not require special permissions beyond contents: read. None of the jobs appear to require write access to the repository, so setting permissions: contents: read at the root is both safe and least-privilege.

Changes required:

  • Insert the following at line 2 (after name: Release):
    permissions:
      contents: read

No other modifications are necessary.


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.
Unable to commit as this autofix suggestion is now outdated
@StuartFerguson StuartFerguson merged commit 7d1ed5a into main Dec 11, 2025
9 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