Skip to content

build: skip Maven download logs #538

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
gcf-merge-on-green merged 1 commit into from
Feb 19, 2021
Merged

build: skip Maven download logs #538

gcf-merge-on-green merged 1 commit into from
Feb 19, 2021

Conversation

@yoshi-automation
Copy link
Contributor

@yoshi-automation yoshi-automation commented Feb 18, 2021

This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/9220b79b-a945-4af3-acda-a18d81d8e09a/targets

  • To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@1aeca92

@yoshi-automation
fix: don't log downloads
b0b9e7f 
@chingor13 This change keeps Maven 3.6.1 and later from spamming our CI logs with page after page of lists of artifacts it's downloading that makes it much harder to find the actual test output.

Source-Author: Elliotte Rusty Harold <elharo@users.noreply.github.com>
Source-Date: Thu Feb 18 19:58:59 2021 +0000
Source-Repo: googleapis/synthtool
Source-Sha: 1aeca92e4a38f47134cb955f52ea76f84f09ff88
Source-Link: googleapis/synthtool@1aeca92
@yoshi-automation yoshi-automation requested a review from a team as a code owner February 18, 2021 23:00
@yoshi-automation yoshi-automation requested a review from a team February 18, 2021 23:00
@yoshi-automation yoshi-automation requested a review from a team as a code owner February 18, 2021 23:00
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Feb 18, 2021
@product-auto-label product-auto-label bot added the api: firestore Issues related to the googleapis/java-firestore API. label Feb 18, 2021
@codecov
Copy link

codecov bot commented Feb 18, 2021 •
edited
Loading

Codecov Report

Merging #538 (b0b9e7f) into master (55dbe00) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #538   +/-   ##
=========================================
  Coverage     73.86%   73.86%           
+ Complexity     1098     1089    -9     
=========================================
  Files            67       67           
  Lines          5840     5840           
  Branches        715      717    +2     
=========================================
  Hits           4314     4314           
  Misses         1302     1302           
  Partials        224      224

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 55dbe00...b0b9e7f. Read the comment docs.

@chingor13 chingor13 changed the title fix: don't log downloads build: skip Maven download logs Feb 18, 2021
@chingor13 chingor13 added the automerge Merge the pull request once unit tests and other checks pass. label Feb 19, 2021
@gcf-merge-on-green gcf-merge-on-green bot merged commit d39f935 into master Feb 19, 2021
@gcf-merge-on-green gcf-merge-on-green bot deleted the autosynth-synthtool branch February 19, 2021 00:18
@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 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chingor13 chingor13 chingor13 approved these changes

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@yoshi-automation @chingor13