Re: Geode Jira/GitHub Integration configuration changes

2017-12-27 Thread Kirk Lund
I'm keeping INFRA-15727 to confirm that the changes have also been made to the geode-native and geode-examples repos. On Wed, Dec 27, 2017 at 3:41 PM, Kirk Lund wrote: > Karen just submitted a PR (#1208). It posted a nice short comment about > the PR to GEODE-4170 without diffs. Looks good to me

Re: Geode Jira/GitHub Integration configuration changes

2017-12-27 Thread Kirk Lund
Karen just submitted a PR (#1208). It posted a nice short comment about the PR to GEODE-4170 without diffs. Looks good to me! I'm going to close INFRA-15727 as fixed. If anyone notices anything not quite right, please let me know and provide a URL. Thanks, Kirk On Wed, Dec 27, 2017 at 3:12 PM, D

Re: Geode Jira/GitHub Integration configuration changes

2017-12-27 Thread Dave Barnes
I must have just missed the cutoff - my PR (#1207), which I created at 2:48, generated diffs in the JIRA. On Wed, Dec 27, 2017 at 3:06 PM, Kirk Lund wrote: > INFRA has made the change. Does anyone have a PR ready to submit so I can > confirm the behavior on a Jira ticket? > > On Wed, Dec 27, 201

Re: Geode Jira/GitHub Integration configuration changes

2017-12-27 Thread Kirk Lund
INFRA has made the change. Does anyone have a PR ready to submit so I can confirm the behavior on a Jira ticket? On Wed, Dec 27, 2017 at 12:57 PM, Kirk Lund wrote: > I filed a request with INFRA to exclude pull request diffs from Jira > comments. > > https://issues.apache.org/jira/browse/INFRA-1

Re: Geode Jira/GitHub Integration configuration changes

2017-12-27 Thread Kirk Lund
I filed a request with INFRA to exclude pull request diffs from Jira comments. https://issues.apache.org/jira/browse/INFRA-15727 On Mon, Dec 18, 2017 at 12:33 PM, William Markito Oliveira < mark...@apache.org> wrote: > +1 for removing the diff - Link to the PR should be good enough. > > On Mon,

Re: Geode Jira/GitHub Integration configuration changes

2017-12-18 Thread William Markito Oliveira
+1 for removing the diff - Link to the PR should be good enough. On Mon, Dec 18, 2017 at 1:18 PM, Jacob Barrett wrote: > +1 for reducing noise! > > On Mon, Dec 18, 2017 at 10:03 AM Joey McAllister > wrote: > > > (Er, from JIRA ticket comments, that is.) > > > > On Mon, Dec 18, 2017 at 10:02 AM

Re: Geode Jira/GitHub Integration configuration changes

2017-12-18 Thread Jacob Barrett
+1 for reducing noise! On Mon, Dec 18, 2017 at 10:03 AM Joey McAllister wrote: > (Er, from JIRA ticket comments, that is.) > > On Mon, Dec 18, 2017 at 10:02 AM Joey McAllister > wrote: > > > +1 to removing the diffs from JIRA tickets. > > > > On Mon, Dec 18, 2017 at 9:58 AM Dan Smith wrote: >

Re: Geode Jira/GitHub Integration configuration changes

2017-12-18 Thread Joey McAllister
(Er, from JIRA ticket comments, that is.) On Mon, Dec 18, 2017 at 10:02 AM Joey McAllister wrote: > +1 to removing the diffs from JIRA tickets. > > On Mon, Dec 18, 2017 at 9:58 AM Dan Smith wrote: > >> Hi Kirk, >> >> I think this is will probably require a request to infra. You can file a >> JI

Re: Geode Jira/GitHub Integration configuration changes

2017-12-18 Thread Joey McAllister
+1 to removing the diffs from JIRA tickets. On Mon, Dec 18, 2017 at 9:58 AM Dan Smith wrote: > Hi Kirk, > > I think this is will probably require a request to infra. You can file a > JIRA with infra assuming we have consensus to get rid of the diffs (huge +1 > from me:) > > Also, please don't cr

Re: Geode Jira/GitHub Integration configuration changes

2017-12-18 Thread Dan Smith
Hi Kirk, I think this is will probably require a request to infra. You can file a JIRA with infra assuming we have consensus to get rid of the diffs (huge +1 from me:) Also, please don't cross post with private@geode. The private list should *only* be used for things that can't be discussed publi