Skip to content

ci: refer base branch name#830

Merged
kenhys merged 1 commit intofluent:fluent-package-v5from
kenhys:ci-ref-base
May 12, 2025
Merged

ci: refer base branch name#830
kenhys merged 1 commit intofluent:fluent-package-v5from
kenhys:ci-ref-base

Conversation

@kenhys
Copy link
Contributor

@kenhys kenhys commented May 12, 2025

When kicked from LTS branch, base branch should be set explicitly, otherwise PR will be created against main branch.

When kicked from LTS branch, base branch should be
set explicitly, otherwise PR will be created against main branch.

Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
@kenhys kenhys requested review from Watson1978 and daipom May 12, 2025 02:51
@kenhys kenhys merged commit 6e91ae8 into fluent:fluent-package-v5 May 12, 2025
115 of 116 checks passed
@kenhys kenhys deleted the ci-ref-base branch May 12, 2025 02:53
@Watson1978 Watson1978 added the backport to v6.0 Need to be backported LTS label Nov 10, 2025
Watson1978 pushed a commit that referenced this pull request Nov 10, 2025
When kicked from LTS branch, base branch should be set explicitly,
otherwise PR will be created against main branch.

Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
@Watson1978 Watson1978 added the backported Already backported to LTS branch (e.g. fluent-package-v5) label Nov 10, 2025
Watson1978 added a commit that referenced this pull request Nov 10, 2025
Backport #831
Backport #830

## #831
Try to use $Env:GITHUB_REF_NAME which contains branch name

github.base_ref is empty when executing via workflow_dispatch.

## #830
When kicked from LTS branch, base branch should be set explicitly,
otherwise PR will be created against main branch.

---------

Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Co-authored-by: Kentaro Hayashi <hayashi@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport to v6.0 Need to be backported LTS backported Already backported to LTS branch (e.g. fluent-package-v5)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants