Skip to content

Comments

[ci] chore: add permissions to workflows for content access#69

Merged
JoyboyBrian merged 2 commits intomainfrom
brian/dev
Feb 18, 2026
Merged

[ci] chore: add permissions to workflows for content access#69
JoyboyBrian merged 2 commits intomainfrom
brian/dev

Conversation

@JoyboyBrian
Copy link
Contributor

@JoyboyBrian JoyboyBrian commented Feb 18, 2026

What

Why

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Refactor (code change that neither fixes a bug nor adds a feature)
  • Documentation update
  • CI/build configuration change

How to Test

Checklist

  • PR title follows [module] type: description format
  • Appropriate labels added (e.g. enhancement, bug, breaking)
  • ruff check . and ruff format --check . pass
  • pyright osmosis_ai/ passes
  • pytest passes (new tests added if applicable)
  • Public API changes are documented
  • No secrets or credentials included

Summary by cubic

Add explicit contents: read permission to the check-pr-title and tests workflows so they can read repository files with the default GITHUB_TOKEN. This follows least-privilege defaults and prevents failures from missing content access.

Written for commit d03da94. Summary will update on new commits.

@github-actions github-actions bot added chore Maintenance/chores ci CI/CD related labels Feb 18, 2026
@JoyboyBrian JoyboyBrian removed the request for review from BaiqingL February 18, 2026 21:18
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@JoyboyBrian JoyboyBrian merged commit b1cfa8c into main Feb 18, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Maintenance/chores ci CI/CD related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant