Skip to content

Update pytest.yaml and fix sync to insights-on-prem repo#119

Merged
joselsegura merged 1 commit into
RedHatInsights:masterfrom
joselsegura:update-sync
Apr 29, 2026
Merged

Update pytest.yaml and fix sync to insights-on-prem repo#119
joselsegura merged 1 commit into
RedHatInsights:masterfrom
joselsegura:update-sync

Conversation

@joselsegura
Copy link
Copy Markdown
Contributor

Description

Add permission section to pytest.yaml and remove some sync to insights-on-prem repository that doesn't make sense at the moment

Type of change

  • Configuration update

Testing steps

Tested locally

@joselsegura joselsegura requested a review from a team as a code owner April 29, 2026 09:34
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 29, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: 2fb61dda-7908-46c4-a3c8-955d578ff695

📥 Commits

Reviewing files that changed from the base of the PR and between bcc0c13 and f093329.

📒 Files selected for processing (2)
  • .github/sync.yml
  • workflows_examples/pytests.yaml
💤 Files with no reviewable changes (1)
  • .github/sync.yml

Summary by CodeRabbit

  • Chores
    • Removed automatic synchronization of the pytest workflow template for a specific repository entry.
    • Added explicit read-only repository permissions to the pytest workflow for enhanced security.

Walkthrough

The changes modify GitHub workflow configuration by removing synchronization of a pytest workflow template for a specific repository entry and adding explicit read-only permissions to the shared pytest workflow template.

Changes

Cohort / File(s) Summary
Workflow Synchronization
.github/sync.yml
Removed the sync entry for RedHatInsights/insights-on-prem repository's pytests.yaml workflow template, disabling automatic updates for that repository.
Workflow Permissions
workflows_examples/pytests.yaml
Added a top-level permissions block with read-only repository access (contents: read) to restrict job-scoped permissions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main changes: updating pytest.yaml with permission section and removing sync configuration for insights-on-prem repo.
Description check ✅ Passed The description clearly explains both changes: adding permissions to pytest.yaml and removing sync configuration that doesn't align with current needs.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Review rate limit: 9/10 reviews remaining, refill in 6 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

@joselsegura joselsegura enabled auto-merge April 29, 2026 09:37
@joselsegura joselsegura merged commit 244dcd0 into RedHatInsights:master Apr 29, 2026
3 checks passed
@joselsegura joselsegura deleted the update-sync branch April 29, 2026 09:47
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