Skip to content

chore(ci): update permissions for pages action - #115

Merged
Orkuncakilkaya merged 1 commit into
mainfrom
inter-2566-permissions-for-docs-action
Oct 1, 2026
Merged

Orkuncakilkaya merged 1 commit into
mainfrom
inter-2566-permissions-for-docs-action

Conversation

@Orkuncakilkaya

Copy link
Copy Markdown
Contributor

No description provided.

@Orkuncakilkaya
Orkuncakilkaya requested a balanced review from Copilot October 1, 2026 09:56
@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

⚠️ This PR doesn't contain any changesets. If there are user-facing changes, don't forget to run:

pnpm exec changeset

to create a changeset.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The permissions are correctly scoped to the job and match the reusable workflow’s requirements.

Review effort: Balanced
Findings: None

What changed in this PR

Updates permissions for the reusable documentation and coverage workflow to support GitHub Pages deployment.

Changes:

  • Grants read access to repository contents.
  • Grants GitHub Pages and OIDC deployment permissions.
File Description
.github/​workflows/​docs-and-coverage.yml Adds required permissions to the reusable workflow job.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Coverage report

St.❔
Category Percentage Covered / Total
🟢 Statements 100% 40/40
🟢 Branches 100% 5/5
🟢 Functions 100% 6/6
🟢 Lines 100% 37/37

Test suite run success

14 tests passing in 5 suites.

Report generated by 🧪jest coverage report action from 99a98b0

Show full coverage report
St File % Stmts % Branch % Funcs % Lines Uncovered Line #s
🟢 All files 100 100 100 100
🟢  src 100 100 100 100
🟢   public-api.ts 100 100 100 100
🟢  src/lib 100 100 100 100
🟢   fingerprint.module.ts 100 100 100 100
🟢   fingerprint.providers.ts 100 100 100 100
🟢   fingerprint.service.ts 100 100 100 100
🟢   version.ts 100 100 100 100
🟢  src/lib/tokens 100 100 100 100
🟢   fingerprint-angular-settings-token.ts 100 100 100 100

@JuroUhlar
JuroUhlar marked this pull request as ready for review October 1, 2026 10:10
@Orkuncakilkaya
Orkuncakilkaya merged commit 3408a95 into main Oct 1, 2026
19 checks passed
@Orkuncakilkaya
Orkuncakilkaya deleted the inter-2566-permissions-for-docs-action branch October 1, 2026 10:16
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