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

Minor cleanup - #329

Merged
software-dov merged 1 commit into
googleapis:masterfrom
software-dov:cleanups
Feb 28, 2020
Merged

software-dov merged 1 commit into
googleapis:masterfrom
software-dov:cleanups

Conversation

@software-dov

Copy link
Copy Markdown
Contributor

Remove unused imports in generated client.py and vars in generated test_%service.py

Remove unused imports in generated client.py and vars in generated test_%service.py
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 28, 2020
@codecov

codecov Bot commented Feb 28, 2020 •

Copy link
Copy Markdown

Codecov Report

Merging #329 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #329   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          25     25           
  Lines        1375   1353   -22     
  Branches      284    286    +2     
=====================================
- Hits         1375   1353   -22
Impacted Files Coverage Δ
gapic/schema/wrappers.py 100% <100%> (ø) ⬆️
gapic/utils/cache.py 100% <0%> (ø) ⬆️
gapic/schema/naming.py 100% <0%> (ø) ⬆️
gapic/generator/options.py 100% <0%> (ø) ⬆️
gapic/schema/imp.py 100% <0%> (ø) ⬆️
gapic/schema/api.py 100% <0%> (ø) ⬆️
gapic/samplegen/samplegen.py 100% <0%> (ø) ⬆️
gapic/schema/metadata.py 100% <0%> (ø) ⬆️
gapic/samplegen_utils/types.py 100% <0%> (ø) ⬆️

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 c89e0ba...a06ca02. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants