Skip to content

chore: bump QualityFlow pin to pick up the output-handling fixes - #7269

Open
guyoron1 wants to merge 1 commit into
fullsend-ai:mainfrom
guyoron1:chore/bump-qf-pin-skip-ci
Open

guyoron1 wants to merge 1 commit into
fullsend-ai:mainfrom
guyoron1:chore/bump-qf-pin-skip-ci

Conversation

@guyoron1

@guyoron1 guyoron1 commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

What

Bump the pinned QualityFlow harness SHA in .fullsend/config.yaml from f6311b4 to 572ba47.

Why

Picks up two QualityFlow fixes that change what lands on PRs here:

Safety

  • harness/qualityflow.yaml is byte-identical at both SHAs, so the #sha256= integrity hash is unchanged (verified against the raw URL).
  • f6311b4..572ba47 contains only those two PRs: agent prompts under qualityflow/agents/ and qualityflow-fullsend's own workflow.

@guyoron1
guyoron1 requested a review from a team as a code owner September 14, 2026 06:29
@github-actions

Copy link
Copy Markdown

E2E tests did not run

E2E tests run automatically for org/repo members and collaborators on pull requests.

For other contributors, a maintainer must add the ok-to-test label after the latest push.

See E2E testing guide for details.

1 similar comment
@github-actions

Copy link
Copy Markdown

E2E tests did not run

E2E tests run automatically for org/repo members and collaborators on pull requests.

For other contributors, a maintainer must add the ok-to-test label after the latest push.

See E2E testing guide for details.

@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Bump QualityFlow pin to restore CI execution

🐞 Bug fix ⚙️ Configuration changes 🕐 Less than 5 minutes

Grey Divider

AI Description

• Pins QualityFlow to templates that no longer append the skip-CI marker.
• Restores CI execution for commits generated on QualityFlow-managed pull requests.
High-Level Assessment

Updating the immutable QualityFlow commit pin is the optimal approach because the unwanted marker was removed upstream. A local template override would duplicate upstream configuration and create unnecessary maintenance risk.

Files changed (1) +1 / -1

Other (1) +1 / -1
config.yamlPin QualityFlow to templates without the skip-CI marker +1/-1

Pin QualityFlow to templates without the skip-CI marker

• Updates the QualityFlow harness source from commit 'f6311b4' to '71aaef50', incorporating the upstream removal of the skip-CI marker. The SHA-256 integrity hash remains unchanged because the referenced harness file is byte-identical.

.fullsend/config.yaml

@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can ask Qodo to dismiss a finding you disagree with, with your reason on record

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@codecov

codecov Bot commented Sep 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

QualityFlow added a skip-ci marker to every commit it pushed, which
suppressed CI on the PR head, and it committed its outputs/ working
directory instead of integrating generated tests into the repo's test
suite. Both are fixed upstream in
redhat-community-ai-tools/qualityflow-fullsend#2 and
redhat-community-ai-tools/qualityflow-fullsend#3.

Bump the pinned harness SHA from f6311b4 to 572ba47.
harness/qualityflow.yaml is byte-identical, so the sha256 integrity hash
is unchanged.

Signed-off-by: guy oron <goron@redhat.com>
@guyoron1
guyoron1 force-pushed the chore/bump-qf-pin-skip-ci branch from 3a691c0 to 91bbf96 Compare September 14, 2026 08:40
@guyoron1 guyoron1 changed the title chore: bump QualityFlow pin to pick up the skip-ci-marker removal chore: bump QualityFlow pin to pick up the output-handling fixes Sep 14, 2026
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.

1 participant