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 2021/03/17 14:16:04 UTC

[GitHub] [ratis-thirdparty] adoroszlai opened a new pull request #13: RATIS-1343. Add basic CI check for ratis-thirdparty

adoroszlai opened a new pull request #13:
URL: https://github.com/apache/ratis-thirdparty/pull/13


   ## What changes were proposed in this pull request?
   
   Add a basic Github Actions check to build the package and run existing smoketests.
   
   https://issues.apache.org/jira/browse/RATIS-1343
   
   ## How was this patch tested?
   
   Push to fork:
   https://github.com/adoroszlai/ratis-thirdparty/actions/runs/661309748
   
   ```
   ...
   [INFO] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0
   ...
   [INFO] BUILD SUCCESS
   ```
   
   Pull request:
   this one


----------------------------------------------------------------
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] [ratis-thirdparty] adoroszlai commented on pull request #13: RATIS-1343. Add basic CI check for ratis-thirdparty

Posted by GitBox <gi...@apache.org>.
adoroszlai commented on pull request #13:
URL: https://github.com/apache/ratis-thirdparty/pull/13#issuecomment-801949070


   Thanks @szetszwo for reviewing and committing it.


----------------------------------------------------------------
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] [ratis-thirdparty] szetszwo merged pull request #13: RATIS-1343. Add basic CI check for ratis-thirdparty

Posted by GitBox <gi...@apache.org>.
szetszwo merged pull request #13:
URL: https://github.com/apache/ratis-thirdparty/pull/13


   


----------------------------------------------------------------
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] [ratis-thirdparty] szetszwo commented on pull request #13: RATIS-1343. Add basic CI check for ratis-thirdparty

Posted by GitBox <gi...@apache.org>.
szetszwo commented on pull request #13:
URL: https://github.com/apache/ratis-thirdparty/pull/13#issuecomment-802437283


   ```
   commit bceb0191706022531a87630011d89e3d4ce4df50 (HEAD -> master, origin/master, origin/HEAD)
   Merge: b085f91 90d8edd
   Author: Tsz-Wo Nicholas Sze <sz...@apache.org>
   Date:   Thu Mar 18 21:54:13 2021 +0800
   
       RATIS-1343. Add basic CI check for ratis-thirdparty. (#13)
   ```
   Somehow the commit is associated with my email address.  Will re-merge 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] [ratis-thirdparty] adoroszlai commented on pull request #13: RATIS-1343. Add basic CI check for ratis-thirdparty

Posted by GitBox <gi...@apache.org>.
adoroszlai commented on pull request #13:
URL: https://github.com/apache/ratis-thirdparty/pull/13#issuecomment-801713819


   @mukul1987 can you please review?


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