Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

Adopt GitHub Actions in gax-java: unit tests and linkage monitor #1062

Merged
merged 5 commits into from May 21, 2020

Conversation

suztomo
Copy link
Member

@suztomo suztomo commented May 21, 2020

@chingor13 @stephaniewang526 Would you review this? By using GitHub Actions, we no longer need to worry about Java 7 build failure: CertPathValidatorException: signature check failed.

Worth noting that "Adopt GitHub Actions in Client Libraries" document is quite useful. Thank you.

This setup worked well for my fork in github.com/suztomo/gax-java:

Screen Shot 2020-05-21 at 11 07 49

suztomo#3

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 21, 2020
@codecov
Copy link

codecov bot commented May 21, 2020

Codecov Report

Merging #1062 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1062   +/-   ##
=========================================
  Coverage     78.75%   78.75%           
  Complexity     1171     1171           
=========================================
  Files           204      204           
  Lines          5173     5173           
  Branches        413      413           
=========================================
  Hits           4074     4074           
  Misses          925      925           
  Partials        174      174           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c62bb8a...0d164aa. Read the comment docs.

@suztomo
Copy link
Member Author

suztomo commented May 21, 2020

@stephaniewang526 @chingor13 Would you merge this? All tests passed.

@chingor13 chingor13 merged commit a8d5ad0 into googleapis:master May 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants