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

samples: add presubmit lint check #258

Merged
merged 2 commits into from
Aug 20, 2020
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/a79f8a17-dbb5-42d4-99e9-73d2b5ffda4d/targets

  • To automatically regenerate this PR, check this box.

Source-Link: googleapis/synthtool@9602086
Source-Link: googleapis/synthtool@c3caf07

…samples

* chore(java_templates): add lint/static analysis presubmit checks for samples

* chore: fix trailing whitespace

Source-Author: Jeff Ching <chingor@google.com>
Source-Date: Mon Aug 17 14:29:16 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: c3caf0704f25a0c365f1c315e804a30b87c62a75
Source-Link: googleapis/synthtool@c3caf07
This was creating too much noise. We will revisit with other options and/or tune these checks.

Source-Author: Jeff Ching <chingor@google.com>
Source-Date: Wed Aug 19 12:26:49 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: 9602086c6c5b05db77950c7f7495a2a3868f3537
Source-Link: googleapis/synthtool@9602086
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 20, 2020
@codecov
Copy link

codecov bot commented Aug 20, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #258   +/-   ##
=========================================
  Coverage     76.54%   76.54%           
  Complexity      502      502           
=========================================
  Files            24       24           
  Lines          3944     3944           
  Branches         48       48           
=========================================
  Hits           3019     3019           
  Misses          841      841           
  Partials         84       84           

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 cd73864...263d77d. Read the comment docs.

@chingor13 chingor13 changed the title [CHANGE ME] Re-generated to pick up changes from synthtool. samples: add presubmit lint check Aug 20, 2020
@chingor13 chingor13 added the automerge Merge the pull request once unit tests and other checks pass. label Aug 20, 2020
@gcf-merge-on-green gcf-merge-on-green bot merged commit e815139 into master Aug 20, 2020
@gcf-merge-on-green gcf-merge-on-green bot deleted the autosynth-synthtool branch August 20, 2020 22:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automerge Merge the pull request once unit tests and other checks pass. 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