Skip to content

Remove unnecessary use of fmt.Sprintf#2819

Merged
gmlewis merged 1 commit intogoogle:masterfrom
testwill:fmt
Jun 26, 2023
Merged

Remove unnecessary use of fmt.Sprintf#2819
gmlewis merged 1 commit intogoogle:masterfrom
testwill:fmt

Conversation

@testwill
Copy link
Copy Markdown
Contributor

unnecessary use of fmt.Sprint
unnecessary use of fmt.Sprintf

Copy link
Copy Markdown
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @testwill !
LGTM.
Merging after tests pass.

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 26, 2023

Codecov Report

Merging #2819 (846c5ce) into master (1e7c685) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #2819   +/-   ##
=======================================
  Coverage   98.05%   98.05%           
=======================================
  Files         136      136           
  Lines       12263    12263           
=======================================
  Hits        12025    12025           
  Misses        162      162           
  Partials       76       76           
Impacted Files Coverage Δ
github/admin_stats.go 100.00% <100.00%> (ø)
github/codespaces.go 92.40% <100.00%> (ø)

@gmlewis gmlewis changed the title chore: unnecessary use of fmt.Sprint Remove unnecessary use of fmt.Sprintf Jun 26, 2023
@gmlewis gmlewis merged commit bed40d1 into google:master Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants