-
Notifications
You must be signed in to change notification settings - Fork 6
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
audit contexts #335
audit contexts #335
Conversation
pkg/v3/flows/conditional_test.go
Outdated
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.
The mock constructors automatically register the *testing.T
and AssertExpectations
on t.Cleanup
.
@@ -1,4 +1,4 @@ | |||
// Code generated by mockery v2.28.1. DO NOT EDIT. | |||
// Code generated by mockery v2.43.2. DO NOT EDIT. |
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.
I didn't mean to use a different version here. Shouldn't CI be failing?
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.
.tool-versions
was on 2.12.1 even though these files were on 2.28.1
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.
Bumped to 2.43.2
e5b8d37
to
d6daff8
Compare
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.
looks good, added a small comment
https://smartcontract-it.atlassian.net/browse/BCF-3204
Supports: