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

ci: linkage monitor CI job to use build.sh#158

Merged
gcf-merge-on-green[bot] merged 1 commit intomasterfrom
autosynth-synthtool
Feb 16, 2021
Merged

ci: linkage monitor CI job to use build.sh#158
gcf-merge-on-green[bot] merged 1 commit intomasterfrom
autosynth-synthtool

Conversation

@yoshi-automation
Copy link
Copy Markdown
Contributor

This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/1e3ec873-bb46-42c7-9fb0-f5bfc36217bf/targets

  • To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@f327d3b

This PR updates the GitHub Actions configuration for Linkage Monitor. Before this change, the CI configuration has "mvn install" command with 6 options. It turned out that a slight difference in the command line option from "build.sh" fails the build in google-auth-library-java repository (GoogleCloudPlatform/cloud-opensource-java#1941). (@Neenu1995 found the build failure. Thanks!)

Therefore, this change updates the configuration so that Linkage Monitor job just uses "build.sh" to install the artifacts into local Maven repository.

# How did I test?

I tested this change in java-spanner repository with this draft PR: googleapis/java-spanner#868

<img width="889" alt="Screen Shot 2021-02-12 at 16 54 18" src="https://user-images.githubusercontent.com/28604/107826467-f3cd5c80-6d52-11eb-86f1-1a8053326978.png">

It succeeded.

Source-Author: Tomo Suzuki <suztomo@google.com>
Source-Date: Tue Feb 16 15:56:09 2021 -0500
Source-Repo: googleapis/synthtool
Source-Sha: f327d3b657a63ae4a8efd7f011a15eacae36b59c
Source-Link: googleapis/synthtool@f327d3b
@yoshi-automation yoshi-automation requested a review from a team February 16, 2021 21:21
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Feb 16, 2021
@product-auto-label product-auto-label Bot added the api: analyticsadmin Issues related to the googleapis/java-analytics-admin API. label Feb 16, 2021
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 16, 2021

Codecov Report

Merging #158 (bb58508) into master (f04c1ac) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #158   +/-   ##
=========================================
  Coverage     84.75%   84.75%           
  Complexity      349      349           
=========================================
  Files             6        6           
  Lines          2564     2564           
  Branches         34       34           
=========================================
  Hits           2173     2173           
  Misses          341      341           
  Partials         50       50           

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 f04c1ac...1d48d00. Read the comment docs.

@Neenu1995 Neenu1995 added the automerge Merge the pull request once unit tests and other checks pass. label Feb 16, 2021
@gcf-merge-on-green gcf-merge-on-green Bot merged commit ef894b9 into master Feb 16, 2021
@gcf-merge-on-green gcf-merge-on-green Bot deleted the autosynth-synthtool branch February 16, 2021 22:20
@gcf-merge-on-green gcf-merge-on-green Bot removed the automerge Merge the pull request once unit tests and other checks pass. label Feb 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: analyticsadmin Issues related to the googleapis/java-analytics-admin API. cla: yes This human has signed the Contributor License Agreement. context: full

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants