Skip to content

Add explicit workflow permissions - #16

Merged
ashwin1111 merged 1 commit into
masterfrom
add-permissions-asdh
Jul 27, 2026
Merged

Add explicit workflow permissions#16
ashwin1111 merged 1 commit into
masterfrom
add-permissions-asdh

Conversation

@ashwin1111

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings July 27, 2026 06:05
@github-actions

Copy link
Copy Markdown
Test Coverage % values
Statements 100% ( 10 / 10 )
Branches 100% ( 1 / 1 )
Functions 100% ( 3 / 3 )
Lines 10% ( 9 / 10 )

@github-actions

Copy link
Copy Markdown

🔁 Code Duplication Report - JavaScript

Format Files analyzed Total lines Total tokens Clones found Duplicated lines Duplicated tokens
json 4 399 2462 1 17 (4.26%) -18.41% 🚀 95 (3.86%) -18.44% 🚀
yaml 9 412 2277 1 33 (8.01%) +0.69% ❗ 199 (8.74%) +0.47% ❗
javascript 1 94 871 0 0 (0.00%) 0 (0.00%)
markdown 1 48 506 0 0 (0.00%) 0 (0.00%)
Total: 15 953 6116 2 50 (5.25%) -2.25% 🚀 294 (4.81%) -2.11% 🚀

🎉 This PR reduces duplicated code by 2.11%!

@github-actions

Copy link
Copy Markdown
End to End Test Coverage % values
Statements 100% ( 10 / 10 )
Branches 100% ( 1 / 1 )
Functions 100% ( 3 / 3 )
Lines 19% ( 9 / 10 )

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds explicit permissions blocks to multiple GitHub Actions workflows to constrain GITHUB_TOKEN access while still allowing workflows to operate (e.g., commenting on PRs).

Changes:

  • Added top-level workflow permissions in several workflows.
  • Adjusted workflow YAML formatting (indent/line cleanup) in secrets-scan.yml and jscpd-duplication-check.yml.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
.github/workflows/test-myself.yml Adds explicit token permissions for the coverage-comment workflow.
.github/workflows/secrets-scan.yml Adds explicit token permissions for gitleaks PR commenting; minor formatting fix.
.github/workflows/original-test.yml Adds explicit token permissions for the coverage-comment workflow.
.github/workflows/jscpd-duplication-check.yml Adds explicit token permissions for the reusable duplication-check workflow; removes trailing blank line.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +7 to +9
permissions:
contents: read
pull-requests: write
Comment on lines +7 to +9
permissions:
contents: read
pull-requests: write
Comment on lines +12 to +15
permissions:
contents: read
pull-requests: write

@ashwin1111
ashwin1111 merged commit 8e9246c into master Jul 27, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants