-
Notifications
You must be signed in to change notification settings - Fork 0
feat: support custom messages; tests: add slash command dispatch #13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
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 adds customizable comment messages to the existing Action, introduces dummy test tasks, and creates a new workflow to dispatch slash commands.
- Define
start-message
,success-message
, andfailure-message
inputs inaction.yml
and update comment bodies to use them with fallbacks. - Add placeholder
test
andtest-fail
tasks topoe_tasks.toml
. - Introduce a new
slash-command-dispatch
workflow to handle issue comment commands.
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
File | Description |
---|---|
poe_tasks.toml | Added dummy test and test-fail tasks |
action.yml | Added custom message inputs and updated comment bodies |
.github/workflows/slash-command-dispatch.yml | New workflow for dispatching slash commands |
Comments suppressed due to low confidence (3)
action.yml:110
- Include a space after
poe
in the default message. For example:('**Running
poe ' + steps.resolve-command.outputs.command + '...**')
to match the original formatting.
> ${{ inputs.start-message || ('**Running `poe' + steps.resolve-command.outputs.command + '`...**') }}
.github/workflows/slash-command-dispatch.yml:8
- [nitpick] Consider using
runs-on: ubuntu-latest
or a supported specific runner (e.g.,ubuntu-22.04
) to ensure compatibility and automatic upgrades.
runs-on: ubuntu-24.04
.github/workflows/slash-command-dispatch.yml:39
- Upgrade to
peter-evans/create-or-update-comment@v4
for consistency with the other steps and to benefit from the latest fixes and features.
uses: peter-evans/create-or-update-comment@v1
|
I hope this works! Everything went great! I'm super happy about it. |
|
No description provided.