Skip to content

Run from main#12

Merged
Yuri05 merged 3 commits into
mainfrom
run_from_main
Apr 28, 2026
Merged

Run from main#12
Yuri05 merged 3 commits into
mainfrom
run_from_main

Conversation

@Yuri05

@Yuri05 Yuri05 commented Apr 28, 2026

Copy link
Copy Markdown
Member

Remove the requirement for creation of extra branches (some users were confused) and allow running the reports directly from the main branch.

Also added a prerequisite in Readme.md (allowing PR-creation by GHA)

Summary by CodeRabbit

  • Improvements

    • Report generation workflows now execute without branch restrictions, allowing evaluation and qualification reports to be created from any branch.
  • Documentation

    • Simplified report creation instructions by removing the separate branch workflow requirement.
    • Added repository setup requirement: enable GitHub Actions permissions for pull request creation and approval.

@coderabbitai

coderabbitai Bot commented Apr 28, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

The pull request removes the check-branch validation job from two GitHub Actions workflows that previously blocked execution on the main branch. The read_inputs job dependency is updated accordingly. Documentation in the README is simplified to remove instructions about creating separate branches for report generation and adds a prerequisite about enabling GitHub Actions permissions.

Changes

Cohort / File(s) Summary
Workflow Branch Validation Removal
.github/workflows/create-evaluation_reports.yml, .github/workflows/create-qualification_reports.yml
Removed check-branch job that previously prevented workflow execution on main branch. Updated read_inputs job to remove dependency on the deleted validation step.
Documentation Updates
README.md
Simplified workflow instructions by removing references to creating and pushing separate branches (e.g., my-reports). Added prerequisite for enabling GitHub Actions repository permissions to create and approve pull requests. Clarified CSV validation focus and adjusted related screenshots.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • Initial commit #1: Both PRs modify the same workflow files; this PR removes the check-branch job and its dependencies that were originally introduced in the related PR.

Suggested reviewers

  • msevestre
  • rwmcintosh

Poem

🐰 No more branch blocks on the main,
The workflows now skip that refrain,
Read inputs flow straight and clear,
Reports hop close without fear,
Simpler paths bring workflow cheer! 🎉

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Run from main' accurately summarizes the main objective of the changeset - removing branch restrictions to allow workflows to execute directly from the main branch.
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch run_from_main

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Line 30: The README image tag is missing an alt attribute which triggers
MD045; update the <img> element by adding a concise descriptive alt text (e.g.,
alt="Screenshot of the app showing X") to improve accessibility and satisfy
linting: locate the <img ...
src="https://github.com/user-attachments/assets/b2c02567-4448-4196-b0bb-1367ab46ecce"
/> entry and add an appropriate alt value summarizing the screenshot content.
- Line 11: The README's image tag (<img
src="https://github.com/user-attachments/assets/b9977f37-e9b4-4a93-8dc2-24f592e2d973">)
is missing an alt attribute causing MD045; update the <img> element to include a
concise descriptive alt text (e.g., alt="Screenshot of XYZ feature") so the tag
has an alt attribute and satisfies accessibility/linting rules.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ce284cbe-fbb3-4c13-9406-ef7b37b5003d

📥 Commits

Reviewing files that changed from the base of the PR and between 0d3b798 and c7516c6.

📒 Files selected for processing (3)
  • .github/workflows/create-evaluation_reports.yml
  • .github/workflows/create-qualification_reports.yml
  • README.md
💤 Files with no reviewable changes (2)
  • .github/workflows/create-qualification_reports.yml
  • .github/workflows/create-evaluation_reports.yml

Comment thread README.md Outdated
Comment thread README.md Outdated
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@Yuri05 Yuri05 merged commit ef97faf into main Apr 28, 2026
3 of 4 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.

2 participants