You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ratis.apache.org by GitBox <gi...@apache.org> on 2020/06/12 17:13:57 UTC
[GitHub] [incubator-ratis] adoroszlai opened a new pull request #124: RATIS-957. Update .asf.yaml to define jira notifications strategy (#116)
adoroszlai opened a new pull request #124:
URL: https://github.com/apache/incubator-ratis/pull/124
## What changes were proposed in this pull request?
Although `.asf.yaml` was recently updated with `notifications` settings (#116), `pull-request-available` label and PR link are not being added to RATIS Jira issues. This PR attempts to fix that.
Config was taken from repos where this works correctly (eg. [Hive](https://github.com/apache/hive/blob/master/.asf.yaml)), as well as from [Ratis current settings](https://gitbox.apache.org/schemes.cgi?incubator-ratis) (although `link label` seems to be [added automatically](https://github.com/apache/infrastructure-puppet/blob/6c5a88281d85d4b61156e21f1775a76d9af04b50/modules/gitbox/files/cgi-bin/schemes.cgi#L55) to that).
https://issues.apache.org/jira/browse/RATIS-957
## How was this patch tested?
Unfortunately there is no easy way to validate before merge.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-ratis] bshashikant commented on pull request #124: RATIS-957. Update .asf.yaml to define jira notifications strategy
Posted by GitBox <gi...@apache.org>.
bshashikant commented on pull request #124:
URL: https://github.com/apache/incubator-ratis/pull/124#issuecomment-643938931
Thanks @adoroszlai for working on this. I have committed this.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-ratis] bshashikant merged pull request #124: RATIS-957. Update .asf.yaml to define jira notifications strategy
Posted by GitBox <gi...@apache.org>.
bshashikant merged pull request #124:
URL: https://github.com/apache/incubator-ratis/pull/124
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-ratis] adoroszlai commented on pull request #124: RATIS-957. Update .asf.yaml to define jira notifications strategy
Posted by GitBox <gi...@apache.org>.
adoroszlai commented on pull request #124:
URL: https://github.com/apache/incubator-ratis/pull/124#issuecomment-643958716
Thanks for @bshashikant reviewing and committing it. Now that it's merged, verified in [RATIS-953](https://issues.apache.org/jira/browse/RATIS-953?page=com.atlassian.jira.plugin.system.issuetabpanels%3Achangehistory-tabpanel): link and worklog were added, but no label (yet?).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org