Skip to content

update repo name in nightly workflow#1485

Merged
lapid92 merged 2 commits intomainfrom
nightly-workflow-reponame
Jul 1, 2025
Merged

update repo name in nightly workflow#1485
lapid92 merged 2 commits intomainfrom
nightly-workflow-reponame

Conversation

@ofirgo
Copy link
Copy Markdown
Contributor

@ofirgo ofirgo commented Jul 1, 2025

Pull Request Description:

Checklist before requesting a review:

  • I set the appropriate labels on the pull request.
  • I have added/updated the release note draft (if necessary).
  • I have updated the documentation to reflect my changes (if necessary).
  • All function and files are well documented.
  • All function and classes have type hints.
  • There is a licenses in all file.
  • The function and variable names are informative.
  • I have checked for code duplications.
  • I have added new unittest (if necessary).

@ofirgo ofirgo requested a review from lapid92 July 1, 2025 13:37
Comment thread .github/workflows/nightly.yml Outdated
Comment on lines 9 to 42

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: {contents: read}

Copilot Autofix

AI 10 months ago

To fix the issue, we will add a permissions block at the root of the workflow file. This block will define the minimal permissions required for the workflow to function correctly. Based on the workflow's operations, it primarily needs read access to repository contents and write access to publish packages. We will set contents: read and packages: write as the permissions.

Suggested changeset 1
.github/workflows/nightly.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/nightly.yml b/.github/workflows/nightly.yml
--- a/.github/workflows/nightly.yml
+++ b/.github/workflows/nightly.yml
@@ -1,2 +1,5 @@
 name: Nightly
+permissions:
+  contents: read
+  packages: write
 on:
EOF
@@ -1,2 +1,5 @@
name: Nightly
permissions:
contents: read
packages: write
on:
Copilot is powered by AI and may make mistakes. Always verify output.
@lapid92 lapid92 merged commit 5ecc21f into main Jul 1, 2025
31 checks passed
@ofirgo ofirgo deleted the nightly-workflow-reponame branch July 2, 2025 12:58
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.

3 participants