-
Notifications
You must be signed in to change notification settings - Fork 892
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
GODRIVER-3018 Use pre-commit for development #1443
Conversation
API Change ReportNo changes found! |
internal/assert/assertions_test.go
Outdated
@@ -1064,7 +1064,6 @@ Diff: | |||
@@ -1,2 +1,2 @@ | |||
-(time.Time) 2020-09-24 00:00:00 +0000 UTC | |||
+(time.Time) 2020-09-25 00:00:00 +0000 UTC | |||
|
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.
This test appears to expect this blank like: https://parsley.mongodb.com/test/mongo_go_driver_v1_tests_42_plus_zlib_zstd_support__version~7.0_os_ssl_40~rhel87_64_go_1_20_test_replicaset_auth_nossl_patch_3a07cb190cee239b59795758a6f26898cb8cb78d_654105853e8e86b87f9395c1_23_10_31_13_47_51/0/125d94d8de41dfe15df278447d79190a?bookmarks=0,10711,24230&shareLine=10713
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.
Thanks, I'll play with the whitespace settings.
(cherry picked from commit b8004e6)
GODRIVER-3018
Summary
Use pre-commit to lint source and text files.
Background & Motivation
Prevent common mistakes and make PR review easier.