Skip to content

net 10 upgrade#79

Merged
StuartFerguson merged 2 commits intomainfrom
task/#77_net_10_upgrade
Dec 7, 2025
Merged

net 10 upgrade#79
StuartFerguson merged 2 commits intomainfrom
task/#77_net_10_upgrade

Conversation

@StuartFerguson
Copy link
Copy Markdown
Member

closes #77
closes #78

Comment thread .github/workflows/createrelease.yml Outdated

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 5 months ago

To address the issue, add a permissions block at the root of the workflow (.github/workflows/createrelease.yml), directly under the workflow name. This block sets the permissions for the GITHUB_TOKEN used by all jobs that do not themselves declare a permissions key. Assign only the least privileges required. For artifact upload/download, only contents: read is needed. As no jobs push code or manipulate issues/pull requests, no elevated permissions are apparent.

Add the following lines to the top of the workflow file, immediately after the workflow name:

permissions:
  contents: read

No imports, new methods, or definitions are needed, as this is a YAML config edit. The change should not affect any workflow functionality but will reduce the privileges granted to the GITHUB_TOKEN in the workflow runs.


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 7b35672 into main Dec 7, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move to Windows Release Workflow Net 10 Upgrade

2 participants