Skip to content

Open Forge PRs ready for review - #40

Merged
GolaraJ merged 1 commit into
mainfrom
feat/forge-ready-for-review-prs
Aug 20, 2026
Merged

Open Forge PRs ready for review#40
GolaraJ merged 1 commit into
mainfrom
feat/forge-ready-for-review-prs

Conversation

@GolaraJ

@GolaraJ GolaraJ commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • require every new Forge PR to pass draft: false explicitly
  • prevent the workflow from relying on the platform draft default
  • add regression coverage for the ready-for-review publication contract

Testing

  • python3 -m unittest discover -s plugins/repo-dreamer/skills/repository-skill-forge/tests -p 'test_*.py' (75 tests)

Upstream sibling PR: https://github.com/github/copilot-dreaming-tasks/pull/28

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: b8cf0691-0fe3-4d67-92ce-ce2a529c400c
@GolaraJ
GolaraJ force-pushed the feat/forge-ready-for-review-prs branch from f5aa924 to 695ee1f Compare August 20, 2026 03:55
@GolaraJ
GolaraJ requested a lite review from Copilot August 20, 2026 03:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the repository-skill-forge publication contract to require explicitly setting draft: false for create actions, and adds a regression test to ensure the contract language remains present.

Changes:

  • Document the requirement to always pass draft: false for Forge create actions so PRs open ready-for-review.
  • Add a unit test that asserts the contract wording is present in SKILL.md.
Show a summary per file
File Description
plugins/repo-dreamer/skills/repository-skill-forge/tests/test_stateless_forge.py Adds a contract regression test that validates SKILL.md contains the new ready-for-review wording.
plugins/repo-dreamer/skills/repository-skill-forge/SKILL.md Documents the explicit draft: false requirement for create actions to ensure PRs are opened ready for review.

Review details

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

@GolaraJ
GolaraJ merged commit e43b0c0 into main Aug 20, 2026
1 check passed
@GolaraJ
GolaraJ deleted the feat/forge-ready-for-review-prs branch August 20, 2026 04:00
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