Skip to content

build: skip Maven download logs #638

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.

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

Merged
merged 1 commit into from
Feb 19, 2021
Merged

Conversation

yoshi-automation
Copy link
Contributor

This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/a6b7617f-b194-4419-a5c2-38ceb3c55e68/targets

  • To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@1aeca92

@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 <[email protected]>
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 review from a team as code owners February 18, 2021 22:51
@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: bigtable Issues related to the googleapis/java-bigtable API. label Feb 18, 2021
@codecov
Copy link

codecov bot commented Feb 18, 2021

Codecov Report

Merging #638 (aab95fd) into master (1cf1fce) will decrease coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #638      +/-   ##
============================================
- Coverage     81.78%   81.72%   -0.06%     
  Complexity     1156     1156              
============================================
  Files           110      110              
  Lines          7214     7214              
  Branches        377      377              
============================================
- Hits           5900     5896       -4     
- Misses         1115     1119       +4     
  Partials        199      199              
Impacted Files Coverage Δ Complexity Δ
...ata/v2/stub/metrics/HeaderTracerUnaryCallable.java 89.47% <0.00%> (-10.53%) 3.00% <0.00%> (ø%)
...om/google/cloud/bigtable/emulator/v2/Emulator.java 59.01% <0.00%> (-1.64%) 14.00% <0.00%> (ø%)

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 1cf1fce...aab95fd. 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 97fafff into master Feb 19, 2021
@gcf-merge-on-green gcf-merge-on-green bot deleted the autosynth-synthtool branch February 19, 2021 00:40
@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
ad548 pushed a commit to ad548/java-bigtable that referenced this pull request Mar 13, 2021
This PR was generated using Autosynth. 🌈

Synth log will be available here:
https://source.cloud.google.com/results/invocations/a6b7617f-b194-4419-a5c2-38ceb3c55e68/targets

- [ ] To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@1aeca92
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigtable Issues related to the googleapis/java-bigtable 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