Skip to content

Add "EVENT_NAME" to Environment Variables#58

Merged
fgrosse merged 8 commits intofgrosse:mainfrom
jm96441n:jmaguire/add-support-for-branch-and-event-type
Mar 11, 2026
Merged

Add "EVENT_NAME" to Environment Variables#58
fgrosse merged 8 commits intofgrosse:mainfrom
jm96441n:jmaguire/add-support-for-branch-and-event-type

Conversation

@jm96441n
Copy link
Copy Markdown
Contributor

@jm96441n jm96441n commented May 22, 2025

Adds EVENT_NAME to environment variables which allows the end user to set what the event name that for the target branch will be. Fixes #46

Also adds event-name and target-branch to inputs with defaults to match current behavior.

Copy link
Copy Markdown
Owner

@fgrosse fgrosse left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the contribution ❤️

Comment thread action.yml Outdated
Copy link
Copy Markdown

@IamNator IamNator left a comment

Choose a reason for hiding this comment

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

LGTM

😎 This should solve my problem.

@jm96441n
Copy link
Copy Markdown
Contributor Author

hey @fgrosse just wanted to check in on this, since I pushed a new commit the original approval no longer applies

@subash-21a
Copy link
Copy Markdown

Hey @fgrosse , Can we merge this, It would be helpful

@jm96441n
Copy link
Copy Markdown
Contributor Author

jm96441n commented Oct 6, 2025

@fgrosse checking back in, can this be approved so it can be merged?

fgrosse added 2 commits March 11, 2026 21:05
- Clarify target-branch description: github.base_ref is only available
  on pull_request events; users on other event types must set it explicitly
- Remove redundant default hint from event-name description
- Add CHANGELOG entry for the event-name and target-branch inputs
@fgrosse
Copy link
Copy Markdown
Owner

fgrosse commented Mar 11, 2026

Hey @jm96441n sorry this PR got lost. I've resolved conflicts now and improved the descriptions just slightly. I will merge this now and release it together with a bunch of other changes as a new version. thanks again for submitting this PR 🙏

@fgrosse fgrosse enabled auto-merge (squash) March 11, 2026 20:14
@fgrosse fgrosse merged commit e12be79 into fgrosse:main Mar 11, 2026
6 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.

feat: support stacked PRs

4 participants