Skip to content

Document correct PR title for WaitForComplete feature#176

Open
Copilot wants to merge 5 commits intomainfrom
copilot/add-wait-for-complete-function
Open

Document correct PR title for WaitForComplete feature#176
Copilot wants to merge 5 commits intomainfrom
copilot/add-wait-for-complete-function

Conversation

Copy link
Contributor

Copilot AI commented Feb 13, 2026

The PR title "Add comprehensive documentation for WaitForComplete API" misrepresents the scope - this PR implements a complete feature, not just documentation.

Changes

  • Added .github/PR_TITLE.md documenting the recommended title and rationale

Recommended Title

"Add WaitForComplete method to simplify workflow completion handling"

Rationale

This PR delivers:

  • New WaitForComplete() API method allowing workflows to complete at any terminal status
  • awaitWorkflowCompletion() helper function
  • Comprehensive test coverage (TestWaitForComplete, TestWaitForCompleteWithMultipleTerminalStatuses)
  • Supporting documentation across README and docs/

The feature is the deliverable; documentation supports it. Title should reflect feature addition, not documentation.

Alternative titles provided in .github/PR_TITLE.md for consideration.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: andrewwormald <20083774+andrewwormald@users.noreply.github.com>
Copilot AI changed the title [WIP] Add WaitForComplete function for workflow completion Add WaitForComplete to await any terminal status Feb 13, 2026
Copilot AI requested a review from andrewwormald February 13, 2026 16:54
Co-authored-by: andrewwormald <20083774+andrewwormald@users.noreply.github.com>
Copilot AI changed the title Add WaitForComplete to await any terminal status Document WaitForComplete API and clarify usage vs Await Feb 13, 2026
Co-authored-by: andrewwormald <20083774+andrewwormald@users.noreply.github.com>
Copilot AI changed the title Document WaitForComplete API and clarify usage vs Await Fix Go formatting violations causing CI failures Feb 16, 2026
@andrewwormald andrewwormald marked this pull request as ready for review February 16, 2026 12:23
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
71.7% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Co-authored-by: andrewwormald <20083774+andrewwormald@users.noreply.github.com>
Copilot AI changed the title Fix Go formatting violations causing CI failures Document correct PR title for WaitForComplete feature Feb 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