skylark71/mattermost-govet

Mattermost Go Vet

This repository contains mattermost-specific go-vet rules that are used to maintain code consistency in mattermost-server.

Included analyzers

1. equalLenAsserts - check for (require/assert).Equal(t, X, len(Y))
2. inconsistentReceiverName - check for inconsistent receiver names in the methods of a struct
3. license - check the license header
4. openApiSync - check for inconsistencies between OpenAPI spec and the source code
5. structuredLogging - check invalid usage of logging (must use structured logging)
6. tFatal - check invalid usage of t.Fatal assertions (instead of testify methods)
7. apiAuditLogs - check that audit records are properly created in the API layer
8. rawSql - check invalid usage of raw SQL queries instead of using the squirrel lib
9. emptyStrCmp - check for idiomatic empty string comparisons
10. pointerToSlice - check for usage of pointer to slice in function definitions
11. mutexLock - check for cases where a mutex is left locked before returning