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

Replace unneeded back ticks (`) with single quotes (') #38

Closed
telpirion opened this issue Jan 12, 2021 · 1 comment · Fixed by #209
Closed

Replace unneeded back ticks (`) with single quotes (') #38

telpirion opened this issue Jan 12, 2021 · 1 comment · Fixed by #209
Assignees
Labels
api: aiplatform Issues related to the AI Platform API. type: process A process-related concern. May include testing, release, or the like.

Comments

@telpirion
Copy link
Contributor

Much of the output from these samples contain unnecessary backticks for string output. We should replace the unneeded backticks with single quotes.

@sofisl sofisl added the type: process A process-related concern. May include testing, release, or the like. label Jan 13, 2021
@JustinBeckwith
Copy link
Contributor

I'm surprised the linter didn't catch those ...

@product-auto-label product-auto-label bot added the api: aiplatform Issues related to the AI Platform API. label Apr 13, 2021
@aribray aribray self-assigned this Jul 15, 2021
@nicain nicain assigned orionnye and unassigned aribray Sep 28, 2021
orionnye added a commit that referenced this issue Oct 6, 2021
orionnye added a commit that referenced this issue Oct 6, 2021

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…ines
orionnye added a commit that referenced this issue Oct 6, 2021

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
nicain added a commit that referenced this issue Oct 21, 2021
* removed backticks being used where a variable could just be logged.

* fix #38 Removed backticks being used where a variable could just be logged.

* fix #38

* fix #38 Joined console Logs of results, that were spread across two lines

* fix #38 making certain all logs had similar spacing

Co-authored-by: nicain <nicain.seattle@gmail.com>
Co-authored-by: Benjamin E. Coe <bencoe@google.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: aiplatform Issues related to the AI Platform API. type: process A process-related concern. May include testing, release, or the like.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants