Skip to content
This repository was archived by the owner on Apr 7, 2026. It is now read-only.

build: skip Maven download logs#891

Merged
chingor13 merged 1 commit intomasterfrom
autosynth-synthtool
Feb 22, 2021
Merged

build: skip Maven download logs#891
chingor13 merged 1 commit intomasterfrom
autosynth-synthtool

Conversation

@yoshi-automation
Copy link
Copy Markdown
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/8440e2da-528d-412a-bf89-385b5618d6dd/targets

  • To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@1aeca92

@yoshi-automation
fix: don't log downloads
0472e8d 
@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 February 18, 2021 22:28
@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: spanner Issues related to the googleapis/java-spanner API. label Feb 18, 2021
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 18, 2021 •
edited
Loading

Codecov Report

Merging #891 (1290fe9) into master (864f2a2) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #891      +/-   ##
============================================
+ Coverage     85.19%   85.21%   +0.02%     
- Complexity     2621     2622       +1     
============================================
  Files           145      145              
  Lines         14262    14262              
  Branches       1378     1378              
============================================
+ Hits          12151    12154       +3     
+ Misses         1537     1535       -2     
+ Partials        574      573       -1
Impacted Files Coverage Δ Complexity Δ
...ain/java/com/google/cloud/spanner/SessionPool.java 89.23% <0.00%> (+0.19%) 73.00% <0.00%> (+1.00%)
...m/google/cloud/spanner/connection/SpannerPool.java 87.89% <0.00%> (+0.52%) 33.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 864f2a2...0472e8d. 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
Copy link
Copy Markdown
Contributor

gcf-merge-on-green Bot commented Feb 19, 2021

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@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
@chingor13 chingor13 merged commit 9a59000 into master Feb 22, 2021
@chingor13 chingor13 deleted the autosynth-synthtool branch February 22, 2021 17:12
ansh0l pushed a commit to ansh0l/java-spanner that referenced this pull request Nov 10, 2022
@codyoss
feat: generate alias for asset betas (googleapis#891)
51c1cc9
rajatbhatta pushed a commit to rajatbhatta/java-spanner that referenced this pull request Nov 17, 2022
@renovate-bot
chore(deps): update dependency com.google.cloud:google-cloud-spanner-…
6d9ebc7 
…jdbc to v2.7.3 (googleapis#891)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@chingor13 chingor13 chingor13 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

api: spanner Issues related to the googleapis/java-spanner 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