Skip to content

fix: define output explicitly#55

Merged
wadackel merged 1 commit into
mainfrom
fix/define-output
May 15, 2025
Merged

fix: define output explicitly#55
wadackel merged 1 commit into
mainfrom
fix/define-output

Conversation

@sisisin

@sisisin sisisin commented May 15, 2025

Copy link
Copy Markdown
Member

issue_number and command were already emitted internally, but they weren’t declared in the outputs section of action.yaml , so I added explicit definitions for them.

@sisisin sisisin self-assigned this May 15, 2025
@sisisin sisisin requested a review from wadackel May 15, 2025 01:44

@wadackel wadackel left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for your fixing, LGTM :)

@wadackel wadackel merged commit 09e8901 into main May 15, 2025
3 checks passed
@wadackel wadackel deleted the fix/define-output branch May 15, 2025 01:47
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.1.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants