-
Notifications
You must be signed in to change notification settings - Fork 0
Update 8hobbies/workflows digest to a18577b #243
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@951751b0bff51b137e19de492856b6405a1ad610 | ||
| uses: 8hobbies/workflows/.github/workflows/npm-lint.yml@a18577bc6b2ba0ee90caea4c978504a835d07977 |
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, add a permissions block to the workflow to explicitly limit the permissions granted to the GITHUB_TOKEN. Since this is a lint job, it likely only needs read access to the repository contents. The best way to fix this is to add a permissions block at the root level of the workflow (before jobs:), specifying contents: read. This ensures that all jobs in the workflow, including those using reusable workflows, will have only the minimum required permissions. Edit .github/workflows/lint.yml to insert the following block after the name: line and before the on: block:
permissions:
contents: readNo additional imports or definitions are needed.
-
Copy modified lines R17-R19
| @@ -14,6 +14,9 @@ | ||
|
|
||
| name: Lint | ||
|
|
||
| permissions: | ||
| contents: read | ||
|
|
||
| on: | ||
| push: | ||
| branches: ["master"] |
| jobs: | ||
| run: | ||
| uses: 8hobbies/workflows/.github/workflows/npm-publish-dry-run.yml@951751b0bff51b137e19de492856b6405a1ad610 | ||
| uses: 8hobbies/workflows/.github/workflows/npm-publish-dry-run.yml@a18577bc6b2ba0ee90caea4c978504a835d07977 |
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, add a permissions block at the top level of the workflow file (just after the name: or before/after the on: block). This block should specify the minimal permissions required for the workflow to function. Since this workflow only delegates to a reusable workflow and does not appear to require any write access, the safest minimal starting point is contents: read. If the reusable workflow requires more, you can adjust accordingly, but starting with contents: read is recommended. The change should be made in .github/workflows/publish-dry-run.yml by inserting the permissions: block after the name: line.
-
Copy modified lines R16-R17
| @@ -13,6 +13,8 @@ | ||
| # limitations under the License. | ||
|
|
||
| name: Publish Dry Run | ||
| permissions: | ||
| contents: read | ||
|
|
||
| on: | ||
| push: |
| jobs: | ||
| test: | ||
| uses: 8hobbies/workflows/.github/workflows/npm-runtime.yml@951751b0bff51b137e19de492856b6405a1ad610 | ||
| uses: 8hobbies/workflows/.github/workflows/npm-runtime.yml@a18577bc6b2ba0ee90caea4c978504a835d07977 |
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, add a permissions block to the workflow file. This can be done at the top level (applies to all jobs) or at the job level (applies to a specific job). Since this workflow only defines a single job (test), either location is acceptable, but adding it at the top level is clearer and future-proof. The minimal safe starting point is permissions: {} (no permissions), but if the reusable workflow requires specific permissions, you should grant only those. Since we do not know the exact requirements of the called workflow, the best fix is to add permissions: {} at the top level, just after the name: key and before on:.
-
Copy modified line R16
| @@ -13,6 +13,7 @@ | ||
| # limitations under the License. | ||
|
|
||
| name: Runtime | ||
| permissions: {} | ||
|
|
||
| on: | ||
| push: |
This PR contains the following updates:
951751b->a18577bConfiguration
📅 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.