[GitHub] geode issue #508: GEODE-2908: Adding the new tag as per swagger notification...

2017-05-15 Thread davinash
Github user davinash commented on the issue: https://github.com/apache/geode/pull/508 @prasi-in Can you please fix the imports. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature e

[GitHub] geode issue #508: GEODE-2908: Adding the new tag as per swagger notification...

2017-05-15 Thread prasi-in
Github user prasi-in commented on the issue: https://github.com/apache/geode/pull/508 Fixed the imports. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] geode issue #508: GEODE-2908: Adding the new tag as per swagger notification...

2017-06-06 Thread metatype
Github user metatype commented on the issue: https://github.com/apache/geode/pull/508 @jaredjstewart any thoughts on this change? If it's good I'd like to merge it since we need it for 1.2.0. --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] geode issue #508: GEODE-2908: Adding the new tag as per swagger notification...

2017-06-06 Thread jaredjstewart
Github user jaredjstewart commented on the issue: https://github.com/apache/geode/pull/508 Looks good to me. I'll merge it in shortly and cherry pick onto the release branch. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] geode issue #508: GEODE-2908: Adding the new tag as per swagger notification...

2017-06-06 Thread jaredjstewart
Github user jaredjstewart commented on the issue: https://github.com/apache/geode/pull/508 Merged as `ac404ad`. Thank you for your contribution! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not ha

[GitHub] geode issue #508: GEODE-2908: Adding the new tag as per swagger notification...

2017-06-07 Thread jaredjstewart
Github user jaredjstewart commented on the issue: https://github.com/apache/geode/pull/508 This PR has already been merged in, but feel free to open a new ticket for that. We unfortunately still have a lot of other "gemfire" references scattered around. --- If your project is set u

[GitHub] geode issue #508: GEODE-2908: Adding the new tag as per swagger notification...

2017-06-07 Thread davebarnes97
Github user davebarnes97 commented on the issue: https://github.com/apache/geode/pull/508 Will add my comment to GEODE-3035 Rest: Swagger UI has lot of references to GemFire, instead they should be changed to geode --- If your project is set up for it, you can reply to this email and