Skip to content

Comments

fix: improve pr detection#62

Merged
bosbaber merged 13 commits intostagingfrom
stephan/intorg-416
Feb 20, 2026
Merged

fix: improve pr detection#62
bosbaber merged 13 commits intostagingfrom
stephan/intorg-416

Conversation

@bosbaber
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings February 20, 2026 21:03
@bosbaber bosbaber merged commit 44b1204 into staging Feb 20, 2026
1 check passed
@bosbaber bosbaber deleted the stephan/intorg-416 branch February 20, 2026 21:05
Copy link

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 pull request improves the PR detection logic in the staging merge workflow by reordering and clarifying the detection process. The workflow now checks the GitHub API for associated PRs first, then validates merge commit patterns, making the logic more efficient and maintainable. The MDX blog file change appears to be unintentional (adding a double period).

Changes:

  • Refactored PR detection logic in the staging merge workflow to check API first, then commit message pattern
  • Renamed variable from mergedIntoStaging to mergedPRIntoStaging for clarity
  • Added validation that merge commits must have multiple parents
  • Unintentionally added a double period to a blog post

Reviewed changes

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

File Description
.github/workflows/staging-merge.yml Refactored PR merge detection logic: reordered API check before commit message check, improved variable naming, and added parent count validation for merge commits
src/content/blog/2026-01-29-interledger-foundation-awards-200000-social-web-foundation-support-decentralized-social-media.mdx Unintentional change: added an extra period at the end of the concluding paragraph

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