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.

Sign up for GitHub

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

Jump to bottom

add missing argument to log statement #895

Merged

kiview merged 1 commit into testcontainers:master from atomfrede:fix-missing-log-argument

Oct 3, 2018

Contributor

atomfrede commented Oct 2, 2018

While doing #894 sonar pointed me to some warnings. I fixed the easy ones here to keep the mr focused, clear and small.


          add missing argument to log statement

42a76e1

atomfrede requested review from bsideup, kiview and rnorth as code owners

October 2, 2018 18:58

kiview approved these changes

View reviewed changes

Member

kiview left a comment

Very nice, maybe there are some more spots in the code where this happens 😅

kiview merged commit c05ad88 into testcontainers:master

kiview added this to the next milestone

kiview added the type/housekeeping label

dbyron0 referenced this pull request in locationlabs/testcontainers-java


          Add missing argument to log statement (#895)

4c74c3e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment