Skip to content

output: add String() (string, error)#30

Merged
bobheadxi merged 1 commit into
mainfrom
05-20-output_add_String_string_error_
May 20, 2022
Merged

output: add String() (string, error)#30
bobheadxi merged 1 commit into
mainfrom
05-20-output_add_String_string_error_

Conversation

@bobheadxi

@bobheadxi bobheadxi commented May 20, 2022

Copy link
Copy Markdown
Member

Closes #28 by adding a (more efficient) alias for the common pattern of getting Lines() and joining it back into a single string.

@bobheadxi

bobheadxi commented May 20, 2022

Copy link
Copy Markdown
Member Author

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@bobheadxi
bobheadxi requested review from a team and jhchabran May 20, 2022 17:36
@bobheadxi bobheadxi linked an issue May 20, 2022 that may be closed by this pull request
@bobheadxi bobheadxi mentioned this pull request May 20, 2022
@bobheadxi
bobheadxi force-pushed the 05-20-output_add_String_string_error_ branch from f4273a9 to f0b1bf4 Compare May 20, 2022 20:49
@codecov-commenter

codecov-commenter commented May 20, 2022

Copy link
Copy Markdown

Codecov Report

Merging #30 (9f43e4b) into lift-output-and-aggregator (b020a1b) will increase coverage by 0.14%.
The diff coverage is 80.00%.

@@                      Coverage Diff                       @@
##           lift-output-and-aggregator      #30      +/-   ##
==============================================================
+ Coverage                       69.54%   69.68%   +0.14%     
==============================================================
  Files                               8        8              
  Lines                             348      353       +5     
==============================================================
+ Hits                              242      246       +4     
- Misses                             85       86       +1     
  Partials                           21       21              
Impacted Files Coverage Δ
output_error.go 16.66% <0.00%> (-1.52%) ⬇️
output.go 77.07% <100.00%> (+0.45%) ⬆️

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 b020a1b...9f43e4b. Read the comment docs.

@bobheadxi
bobheadxi force-pushed the 05-20-output_add_String_string_error_ branch from f0b1bf4 to 9f43e4b Compare May 20, 2022 20:54
Base automatically changed from lift-output-and-aggregator to main May 20, 2022 22:32
@bobheadxi
bobheadxi force-pushed the 05-20-output_add_String_string_error_ branch from 9f43e4b to 31d00ba Compare May 20, 2022 22:33
@bobheadxi

Copy link
Copy Markdown
Member Author

Merging this directly as well to clear the way for some refactors!

@bobheadxi
bobheadxi merged commit 585cd79 into main May 20, 2022
@bobheadxi
bobheadxi deleted the 05-20-output_add_String_string_error_ branch May 20, 2022 22:33
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.

add output.String()

2 participants