Skip to content

Commit

Permalink
🤯 Revert "errs: Add Ignore function" (#40)
Browse files Browse the repository at this point in the history
The errs.Ignore function did not end up working the way we wanted it to,
as lint could no longer tell that `Close` was being called on
`Request.Body` since the `Close` function was passed by value to be
executed anonymously.

Since the main purpose was to have code that documented explicitly that
we were ignoring errors and not have lint warnings, and that did not
work, there is no purpose to this code any more.

This reverts commit 1ed685a.

This merges the following commits:
* Revert "errs: Add Ignore function"

     errs/ignore.go      |  9 ---------
     errs/ignore_test.go | 18 ------------------
     2 files changed, 27 deletions(-)

Pull-Request: #40
  • Loading branch information
camh- committed Oct 8, 2020
2 parents 2f3c9cf + 53a5042 commit 71cbe2e
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 27 deletions.
9 changes: 0 additions & 9 deletions errs/ignore.go

This file was deleted.

18 changes: 0 additions & 18 deletions errs/ignore_test.go

This file was deleted.

0 comments on commit 71cbe2e

Please sign in to comment.