-
Notifications
You must be signed in to change notification settings - Fork 0
Update 8hobbies/workflows digest to b4d993f #258
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| jobs: | ||
| lint: | ||
| uses: 8hobbies/workflows/.github/workflows/npm-lint.yml@05401f51a9ee37f416b048394b9f3e4724bbc03f | ||
| uses: 8hobbies/workflows/.github/workflows/npm-lint.yml@b4d993fbc9bbda7422eab6cb21e469bf4ad10562 |
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 3 months ago
To fix the problem, you should add a permissions block at the root level of the workflow file (.github/workflows/lint.yml). This ensures the GITHUB_TOKEN receives only the minimum required privileges (typically just contents: read for a lint workflow). This block should be inserted below the workflow name (name: Lint) but above the on: specification so it's clear and applies globally to all jobs unless further restricted. No other code changes or imports are necessary; simply add the following YAML block:
permissions:
contents: readThis restricts the workflow GITHUB_TOKEN to read repository contents, which is sufficient for a lint workflow and adheres to the principle of least privilege.
-
Copy modified lines R16-R17
| @@ -13,6 +13,8 @@ | ||
| # limitations under the License. | ||
|
|
||
| name: Lint | ||
| permissions: | ||
| contents: read | ||
|
|
||
| on: | ||
| push: |
| jobs: | ||
| run: | ||
| uses: 8hobbies/workflows/.github/workflows/npm-publish-dry-run.yml@05401f51a9ee37f416b048394b9f3e4724bbc03f | ||
| uses: 8hobbies/workflows/.github/workflows/npm-publish-dry-run.yml@b4d993fbc9bbda7422eab6cb21e469bf4ad10562 |
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 3 months ago
To fix the issue, you should add an explicit permissions block to the workflow file. This can be done at the workflow level (top-level, applying to all jobs unless overridden) or at the job level (within jobs.run). Since the job simply invokes a reusable workflow (and the file is otherwise minimal), it is most appropriate and future-proof to add the permissions block at the top level—just after the name or on blocks. The safest minimal set (if the invoked workflow only needs to read the contents) is contents: read; if it needs additional permissions, you can expand as needed. In the absence of specific requirements, contents: read is the recommended starting point.
-
Copy modified lines R17-R19
| @@ -14,6 +14,9 @@ | ||
|
|
||
| name: Publish Dry Run | ||
|
|
||
| permissions: | ||
| contents: read | ||
|
|
||
| on: | ||
| push: | ||
| branches: ["master"] |
| jobs: | ||
| test: | ||
| uses: 8hobbies/workflows/.github/workflows/npm-runtime.yml@05401f51a9ee37f416b048394b9f3e4724bbc03f | ||
| uses: 8hobbies/workflows/.github/workflows/npm-runtime.yml@b4d993fbc9bbda7422eab6cb21e469bf4ad10562 |
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 3 months ago
To resolve the issue, add a permissions block declaring the minimum required GitHub token permissions for the workflow at the top level, or within the job definition. Since there is only one job and it's a directly-used reusable workflow, the best place is at the root level (global for the workflow) after the name: and before on:. If you don't know the minimum permissions needed, you should start with the minimal recommended (e.g. contents: read) and refine later if workflows actually require more. No other changes are needed.
-
Copy modified lines R16-R17
| @@ -13,6 +13,8 @@ | ||
| # limitations under the License. | ||
|
|
||
| name: Runtime | ||
| permissions: | ||
| contents: read | ||
|
|
||
| on: | ||
| push: |
Coming soon: The Renovate bot (GitHub App) will be renamed to Mend. PRs from Renovate will soon appear from 'Mend'. Learn more here.
This PR contains the following updates:
05401f5->b4d993fConfiguration
📅 Schedule: Branch creation - "on Sunday" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.