Skip to content

Remove unnecessary use of fmt.Sprintf - #2819

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

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

Conversation

@testwill

Copy link
Copy Markdown
Contributor

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

@gmlewisgmlewis left a comment

Copy link
Copy Markdown
Collaborator

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

codecovBot commented Jun 26, 2023

Copy link
Copy Markdown

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 FilesCoverage Δ
github/admin_stats.go100.00% <100.00%> (ø)
github/codespaces.go92.40% <100.00%> (ø)

@gmlewisgmlewis changed the title chore: unnecessary use of fmt.SprintRemove unnecessary use of fmt.SprintfJun 26, 2023
@gmlewis
gmlewis merged commit bed40d1 into google:masterJun 26, 2023
jlaportebot added a commit to jlaportebot/go-github that referenced this pull request Jun 28, 2026
Sign up for freeto 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

@testwill@gmlewis