-
Notifications
You must be signed in to change notification settings - Fork 0
Reply message improvement #54
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
…moved from text payload due to bad formated messages.
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 improves reply message handling in the Slack message GitHub action by adding conditional execution and fixing message formatting.
- Added conditional execution to prevent unnecessary processing when no reply is needed
- Fixed message text formatting by removing brackets and update timestamp parameter
- Streamlined the reply message payload structure
Comments suppressed due to low confidence (1)
.github/actions/slack-message/action.yml:124
- The removal of the
update-tsparameter may break existing functionality if this action was being used to update existing messages. Consider documenting this breaking change or ensuring it's intentional.
update-ts: ${{ inputs.update }}
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
…ailed to push 476146385463.dkr.ecr.us-east-1.amazonaws.com/test-templates:latest-custom: unknown: The image tag 'latest-custom' already exists in the 'test-templates' repository and cannot be overwritten because the tag is immutable.
No description provided.