Skip to content

Allow deploy-card to show message provided in options#316

Closed
terrymun wants to merge 2 commits intomikesprague:mainfrom
terrymun:main
Closed

Allow deploy-card to show message provided in options#316
terrymun wants to merge 2 commits intomikesprague:mainfrom
terrymun:main

Conversation

@terrymun
Copy link
Copy Markdown

Hi @mikesprague! Thanks for this amazing package. I was consuming it and realized that the deploy-card and message options are mutually exclusive: if deploy-card: true is used, then the message provided will not be rendered in the card at all.

This pull request allows message to be added to the deploy-card.

@mikesprague
Copy link
Copy Markdown
Owner

Thank you for the PR! And sorry for the (extended) delay in getting back to you and looking at it.

I have finally gotten around to modernizing this codebase some and implementing a couple things - this has included setting up unit tests with 100% coverage as well as the feature in your PR.

This is now available in v1.17.0 (v1 has been updated to same version) and the README has been updated to reflect this so I am going to close this PR.

Thanks again for the submission!

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.

2 participants