Skip to content

fix(aidd-vcs): scope PR verification to changed behavior - #982

Merged
blafourcade merged 2 commits into
nextfrom
codex/fix-pr-verification-scope
Oct 12, 2026
Merged

blafourcade merged 2 commits into
nextfrom
codex/fix-pr-verification-scope

Conversation

@alexsoyes

@alexsoyes alexsoyes commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

🎯 What & why

Keep PR verification focused on the changed feature instead of generic CI results.

🛠️ How it works

Collection summarizes the actual diff. Drafting adds one focused rule: minimal concrete steps, expected results, no generic CI summaries, and explicit labels for unrun scenarios.

🧪 How to verify

Not run in a separate skill session:

  • Ask the updated PR skill to redraft the todo → batch PR using its existing CI results and unrun agent scenario.
    • Expected: only feature-specific steps, each with an expected result.
    • Expected: no generic CI recap.
    • Expected: the unrun scenario remains labeled.

✅ I certify

  • I DO CERTIFY I READ EACH LINE OF THE PULL REQUEST BECAUSE I AM A SOFTWARE ENGINEER, NOT A AI PUPPY.

@alexsoyes
alexsoyes marked this pull request as ready for review October 9, 2026 04:33
@alexsoyes
alexsoyes requested a review from a team as a code owner October 9, 2026 04:33
@alexsoyes

Copy link
Copy Markdown
Contributor Author

J'en avais marre de voir des PRs avec comme étape de vérification des pnpm test... on veut un vrai comportement à checker

@alexsoyes

Copy link
Copy Markdown
Contributor Author

@blafourcade genre cette merde : #981

Comment on a pu laisser ça dans les PRs aussi longtemps, pure non sense :)

@blafourcade
blafourcade merged commit 60aab10 into next Oct 12, 2026
25 checks passed
@blafourcade
blafourcade deleted the codex/fix-pr-verification-scope branch October 12, 2026 04:44
@aidd-bot aidd-bot Bot mentioned this pull request Oct 16, 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.

2 participants