Skip to content

add/changes#433

Open
Micha Hobert (Isengo1989) wants to merge 1 commit into
mainfrom
change/branch-naming
Open

add/changes#433
Micha Hobert (Isengo1989) wants to merge 1 commit into
mainfrom
change/branch-naming

Conversation

@Isengo1989

Copy link
Copy Markdown
Contributor

No description provided.

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 reusable plugin schema-generation workflows so that the “Open a pull request” job targets the branch the workflow run is associated with, avoiding cross-branch collisions by including the base branch name in the generated PR branch name and title.

Changes:

  • Add a BRANCH_NAME env var to capture the workflow run’s branch context.
  • Set create-pull-request base to BRANCH_NAME so PRs target the current branch.
  • Include BRANCH_NAME in the PR branch name and title to keep PRs distinct per base branch.

Reviewed changes

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

File Description
.github/workflows/plugin_digitalsalesroom_schema.yml Targets PR base to the current run branch and namespaces PR branch/title with it.
.github/workflows/plugin_customproducts_schema.yml Same branch-aware PR targeting and naming for Custom Products schema updates.
.github/workflows/plugin_commercial_schema.yml Same branch-aware PR targeting and naming for Commercial schema updates.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +53 to +54
env:
BRANCH_NAME: ${{ github.head_ref || github.ref_name }}
Comment on lines +47 to +48
env:
BRANCH_NAME: ${{ github.head_ref || github.ref_name }}
Comment on lines +51 to +52
env:
BRANCH_NAME: ${{ github.head_ref || github.ref_name }}
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