Skip to content

Fix branch name in workflow#39

Merged
joselsegura merged 2 commits intoRedHatInsights:mainfrom
joselsegura:fix_pipeline
Mar 17, 2026
Merged

Fix branch name in workflow#39
joselsegura merged 2 commits intoRedHatInsights:mainfrom
joselsegura:fix_pipeline

Conversation

@joselsegura
Copy link
Contributor

Small fix in branches where workflow should be run when merged

Type of change

  • Configuration update

Testing steps

N/A

Checklist

  • pre-commit run --all passes
  • updated documentation wherever necessary
  • added or modified tests if necessary
  • updated schemas and validators in insights-data-schemas in case of input/output change

@coderabbitai
Copy link

coderabbitai bot commented Mar 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e3e706c1-40fd-4a4d-b6d9-84ff0b4defb1

📥 Commits

Reviewing files that changed from the base of the PR and between 61acbf3 and 94afcce.

📒 Files selected for processing (1)
  • .github/workflows/linters.yaml

Summary by CodeRabbit

  • Chores
    • Updated GitHub Actions workflow configuration to trigger linting checks on both main and master branches for push events.

Walkthrough

The GitHub Actions linters workflow was updated to trigger on push events for both main and master branches, rather than a single branch. The pull request trigger configuration remains unchanged.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/linters.yaml
Updated push event trigger to include both main and master branches.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title directly and specifically describes the main change: fixing the branch name in the workflow configuration file.
Description check ✅ Passed The description covers the essential sections including Type of change (Configuration update) and Testing steps (N/A), with the checklist partially completed.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
📝 Coding Plan
  • Generate coding plan for human review comments

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

@joselsegura joselsegura enabled auto-merge March 17, 2026 10:05
@joselsegura joselsegura merged commit 0d7c23b into RedHatInsights:main Mar 17, 2026
8 checks passed
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