-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor: remove ContinueWithError #630
Conversation
Remove Continue with error function
Tests passed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggested a small change to keep the behaviour identical
@yoavrotems Please can you revisit the name of this PR? Something like "Refactor: remove ContinueWithError" would be more meaningful to someone else looking at the PR list |
@lizrice I changed but apparently didn't apply my change... |
Co-authored-by: Liz Rice <liz@lizrice.com>
Remove unnecessary ')'
removed fmt.Fprintf(os.Stderr, "%s: %s", cmd.Args, err) since it wasn't suppose to print.
option --no-ri and --no-doc are deprecated we have to use --no-document instead. rubygems/heroku-buildpack-bundler2#1 (comment)
Codecov Report
@@ Coverage Diff @@
## master #630 +/- ##
==========================================
+ Coverage 65.57% 69.41% +3.83%
==========================================
Files 12 12
Lines 1101 1213 +112
==========================================
+ Hits 722 842 +120
+ Misses 339 333 -6
+ Partials 40 38 -2
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Update util.go Remove Continue with error function * Update cmd/util.go Co-authored-by: Liz Rice <liz@lizrice.com> * Update util.go * Update util.go Remove unnecessary ')' * Update util.go removed fmt.Fprintf(os.Stderr, "%s: %s", cmd.Args, err) since it wasn't suppose to print. * Update util.go * Update .travis.yml option --no-ri and --no-doc are deprecated we have to use --no-document instead. rubygems/heroku-buildpack-bundler2#1 (comment) Co-authored-by: Liz Rice <liz@lizrice.com>
Remove Continue with error function