Skip to content

feat: expose lucky output from action#829

Merged
kitsuyui merged 1 commit into
mainfrom
feat/add-lucky-output
May 21, 2026
Merged

feat: expose lucky output from action#829
kitsuyui merged 1 commit into
mainfrom
feat/add-lucky-output

Conversation

@kitsuyui
Copy link
Copy Markdown
Owner

Summary

Expose the lucky boolean as an action output so downstream workflow steps can branch on whether a lucky commit or PR number was found.

  • action.yml: declare outputs.lucky with description
  • src/index.ts: call core.setOutput('lucky', ...) after building the message

Motivation

Previously, callers had to parse PR comments to determine if the action found a lucky commit. With this output, downstream steps can use steps.<id>.outputs.lucky directly.

Test plan

  • Existing tests pass (bun run test)
  • Build succeeds (bun run build)
  • Lint passes (bun run lint)

Declare the `lucky` output in action.yml and emit its value via
`core.setOutput` in src/index.ts so callers can branch on whether a
lucky commit or PR number was detected.
@github-actions
Copy link
Copy Markdown

🎉 Happy commit!

@github-actions
Copy link
Copy Markdown

Code Metrics Report

main (048b9dd) #829 (7ffc5b5) +/-
Coverage 99.0% 99.0% 0.0%
Code to Test Ratio 1:0.5 1:0.5 -0.1
Test Execution Time 1s 2s +1s
Details
  |                     | main (048b9dd) | #829 (7ffc5b5) | +/-  |
  |---------------------|----------------|----------------|------|
  | Coverage            |          99.0% |          99.0% | 0.0% |
  |   Files             |              5 |              5 |    0 |
  |   Lines             |            107 |            107 |    0 |
  |   Covered           |            106 |            106 |    0 |
- | Code to Test Ratio  |          1:0.5 |          1:0.5 | -0.1 |
  |   Code              |           1408 |           1409 |   +1 |
  |   Test              |            777 |            777 |    0 |
- | Test Execution Time |             1s |             2s |  +1s |

Reported by octocov

@github-actions
Copy link
Copy Markdown

gh-counter

PR gate

Removed Added +/-
TODO/FIXME 0 0 0

Repo dashboard

main (048b9dd) #829 (7ffc5b5) +/-
TODO/FIXME 0 0 0

Reported by gh-counter

@kitsuyui kitsuyui merged commit 3677d9a into main May 21, 2026
5 checks passed
@kitsuyui kitsuyui deleted the feat/add-lucky-output branch May 21, 2026 13:34
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.

1 participant