Skip to content

Refine target branch detection in update-studio-docs workflow#25166

Merged
voloagent merged 1 commit into
devfrom
skoc10-patch-3
Mar 26, 2026
Merged

Refine target branch detection in update-studio-docs workflow#25166
voloagent merged 1 commit into
devfrom
skoc10-patch-3

Conversation

@skoc10

@skoc10 skoc10 commented Mar 26, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Updated target branch handling in workflow to allow auto-detection from the latest stable ABP release if not provided.
@voloagent voloagent merged commit c20409d into dev Mar 26, 2026
4 checks passed
@voloagent voloagent deleted the skoc10-patch-3 branch March 26, 2026 04:45

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 refines the update-studio-docs GitHub Actions workflow so the target branch can be left empty and auto-detected based on the latest stable ABP Framework release, instead of defaulting to dev.

Changes:

  • Allow target_branch to be empty and introduce an auto-detection step that derives rel-X.Y from the latest stable ABP release tag.
  • Update downstream steps (validation, checkout, PR creation/summary) to use the resolved target branch output.
  • Add a default reviewer (skoc10) when creating/updating the docs PR.

Comment on lines +77 to +81
RELEASES=$(curl -fsS \
-H "Accept: application/vnd.github+json" \
-H "Authorization: Bearer ${{ secrets.GITHUB_TOKEN }}" \
"https://api.github.com/repos/abpframework/abp/releases?per_page=20")

Copilot AI Mar 26, 2026

Copy link

Choose a reason for hiding this comment

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

This workflow now fetches ABP releases twice (here in Resolve target branch, and later in Fetch latest stable ABP version) with essentially the same filter logic. That’s duplicated logic to maintain, adds an extra API call, and can produce inconsistent results if a new ABP release is published between the two steps. Consider fetching the latest stable ABP version once early and reusing it both to derive TARGET_BRANCH and to set ABP_VERSION later.

Copilot uses AI. Check for mistakes.
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.

3 participants