[GODRIVER-2642] Enable the exportloopref linter Created: 02/Nov/22 Updated: 24/Jan/23 Resolved: 20/Nov/22 |
|
| Status: | Closed |
| Project: | Go Driver |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.12.0, 1.12.0-alpha1 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Benji Rewis (Inactive) | Assignee: | Benji Rewis (Inactive) |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Epic Link: | Improve Testing Suite | ||||||||
| Quarter: | FY23Q4 | ||||||||
| Documentation Changes: | Not Needed | ||||||||
| Description |
|
In parallelizing the test suite, I've found it's very easy to make the mistake of not capturing the for-loop variable when using t.Parallel(). This a really common issue for Go developers and will probably be changed in the language. Until then, we should use a linter like exportloopref to check for this error. |
| Comments |
| Comment by Githook User [ 20/Nov/22 ] |
|
Author: {'name': 'Benjamin Rewis', 'email': '32186188+benjirewis@users.noreply.github.com', 'username': 'benjirewis'}Message: |
| Comment by Benji Rewis (Inactive) [ 17/Nov/22 ] |