Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

bringking
Copy link
Contributor

@bringking bringking commented Nov 30, 2017

We wanted to customize the message in danger with a prefix and saw that we couldn't modify the message format. This PR -

  • Adds a message format function to the options object that is passed the JIRA URls array and the
    selected emoji. The function can return a custom string to format the danger message.

  • Adds a test

  • Updates the example in the readme

Adds a message format function to the options object that is passed the JIRA URls array and the
selected emoji. The function can return a custom string to format the danger message.
Copy link
Owner

@macklinu macklinu left a comment

Choose a reason for hiding this comment

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

This is a good idea. Thanks for the contribution and the test! 😄

@macklinu macklinu merged commit ecfead9 into macklinu:master Nov 30, 2017
@macklinu
Copy link
Owner

Once the master branch build succeeds on Travis, version 1.3.0 should be shipped with this feature. 👍

@bringking
Copy link
Contributor Author

@macklinu awesome man! Thank you for the quick response! 💯

mcgrathg pushed a commit to mcgrathg/danger-plugin-jira-integration that referenced this pull request Apr 13, 2023
# 1.0.0 (2023-04-13)

### Bug Fixes

* **jiraIssue:** properly resolve URL ([af099b9](af099b9))
* **node:** support Node 4 ([d89d651](d89d651))
* only add one example JIRA key to warning ([macklinu#47](https://github.com/mcgrathg/danger-plugin-jira-integration/issues/47)) ([56dee38](56dee38))

### Features

* `key` is now optional field ([2acddae](2acddae))
* **$options:** Adds message format function to options ([macklinu#33](https://github.com/mcgrathg/danger-plugin-jira-integration/issues/33)) ([ecfead9](ecfead9))
* **jiraIssue:** add initial implementation ([#1](#1)) ([8638e2f](8638e2f))
* **options:** add location to options ([macklinu#44](https://github.com/mcgrathg/danger-plugin-jira-integration/issues/44)) ([96e6261](96e6261))
* support multiple JIRA issues in PR title ([86c01d0](86c01d0)), closes [macklinu#8](https://github.com/mcgrathg/danger-plugin-jira-integration/issues/8)
* support multiple JIRA projects ([macklinu#26](https://github.com/mcgrathg/danger-plugin-jira-integration/issues/26)) ([f0a084f](f0a084f)), closes [macklinu#24](https://github.com/mcgrathg/danger-plugin-jira-integration/issues/24)

### BREAKING CHANGES

* **jiraIssue:** this commit introduces functionality ready for a 1.0.0 release.
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.