-
Notifications
You must be signed in to change notification settings - Fork 50
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
Remove Assert Package 3/3 #1205
Conversation
Remove Assert pkg from ./foundation. Refactor is.New(t) to be initialised in each subtest
… be caught by isTrue(res == nil) alone
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.
since this is the last PR to remove the assert package, let's add the package name to the depguard
linter for golangci-lint, to make sure no other file will have it in the future
Line 7 in 31e283d
depguard: |
Why don't we just remove the package itself? Then there's nothing to import, nothing to guard against 🙂 |
@lovromazgon lol totally forgot that we're the ones who created it, let's do what he said 🤣 |
Description
Refactor all
_test.go
that usedpkg/foundation/assert
to use (https://github.com/matryer/is) in the following packages:PR (1/3)
./pipeline
./record
./conduit
./web
PR (2/3)
./processor
./orchestrator
PR (3/3)
./foundation
Partially Fixes #260
Quick checks: