You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2021/03/08 11:27:37 UTC

[GitHub] [geode] mivanac opened a new pull request #6101: Geode 6150 x

mivanac opened a new pull request #6101:
URL: https://github.com/apache/geode/pull/6101


   Thank you for submitting a contribution to Apache Geode.
   
   In order to streamline the review of the contribution we ask you
   to ensure the following steps have been taken:
   
   ### For all changes:
   - [ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?
   
   - [ ] Has your PR been rebased against the latest commit within the target branch (typically `develop`)?
   
   - [ ] Is your initial contribution a single, squashed commit?
   
   - [ ] Does `gradlew build` run cleanly?
   
   - [ ] Have you written or updated unit tests to verify your changes?
   
   - [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)?
   
   ### Note:
   Please ensure that once the PR is submitted, check Concourse for build issues and
   submit an update to your PR as soon as possible. If you need help, please send an
   email to dev@geode.apache.org.
   


----------------------------------------------------------------
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] [geode] lgtm-com[bot] commented on pull request #6101: Geode 6150 x

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #6101:
URL: https://github.com/apache/geode/pull/6101#issuecomment-801445110


   This pull request **fixes 2 alerts** when merging 2a19d5576c50b7cf944e0b3db7fd59a56755e6fa into dfc16562a1965d4d0bc32756bdf2a9664b4127c2 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-1b4f66ce0d5f1fce16ee645818cc20f65d157eff)
   
   **fixed alerts:**
   
   * 2 for Dereferenced variable may be null


----------------------------------------------------------------
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] [geode] mivanac commented on pull request #6101: Geode 6150 x

Posted by GitBox <gi...@apache.org>.
mivanac commented on pull request #6101:
URL: https://github.com/apache/geode/pull/6101#issuecomment-802335067


   New PR created


-- 
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] [geode] mivanac closed pull request #6101: Geode 6150 x

Posted by GitBox <gi...@apache.org>.
mivanac closed pull request #6101:
URL: https://github.com/apache/geode/pull/6101


   


-- 
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] [geode] lgtm-com[bot] commented on pull request #6101: Geode 6150 x

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #6101:
URL: https://github.com/apache/geode/pull/6101#issuecomment-792713492


   This pull request **fixes 1 alert** when merging 2567ef46009ac3e88011b7114f0f4cf61c4e18c8 into df0d3c0271c822ed7614951f36f8d59f35ab0637 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-9a1156a50f716d238731312c3ab2ba203c8fdf08)
   
   **fixed alerts:**
   
   * 1 for Potential input resource leak


----------------------------------------------------------------
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] [geode] lgtm-com[bot] commented on pull request #6101: Geode 6150 x

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #6101:
URL: https://github.com/apache/geode/pull/6101#issuecomment-801485839


   This pull request **fixes 2 alerts** when merging d817fcd24c336502922d798d34ddc3cc5e78a55b into dfc16562a1965d4d0bc32756bdf2a9664b4127c2 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-f7f8fb5f8ce5866251763861f5df85d1c801a985)
   
   **fixed alerts:**
   
   * 2 for Dereferenced variable may be null


----------------------------------------------------------------
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