Skip to content

Comments

Update auto PR merge step to only check for PR Build Status in Regenerate OpenAPI Connectors Workflow#7805

Merged
ThisaruGuruge merged 5 commits intoballerina-platform:mainfrom
chathushkaayash:gen-connector
Apr 18, 2025
Merged

Update auto PR merge step to only check for PR Build Status in Regenerate OpenAPI Connectors Workflow#7805
ThisaruGuruge merged 5 commits intoballerina-platform:mainfrom
chathushkaayash:gen-connector

Conversation

@chathushkaayash
Copy link
Contributor

@ThisaruGuruge ThisaruGuruge requested a review from Copilot April 18, 2025 06:00
Copy link
Contributor

Copilot AI left a comment

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 auto-merge step to only check the PR Build Workflow status during connector regeneration.

  • Replaces the conditional loop with an unconditional while loop for continuous polling.
  • Implements a specific check for the “Run PR Build Workflow / Build” status via gh pr view and JSON parsing.
  • Removes the earlier generic status evaluation logic.
Comments suppressed due to low confidence (1)

.github/workflows/regenerate-connector-template.yml:255

  • [nitpick] The sleep statement following the loop exit may be redundant and could delay subsequent steps unnecessarily. Consider reviewing if this delay is required after a successful build status check.
            sleep 20

@ThisaruGuruge ThisaruGuruge merged commit a92354c into ballerina-platform:main Apr 18, 2025
2 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.

Connector Regeneration Workflow Should Only Check the PR Build Status

2 participants