1fd02a13c2
* Install `buf` instead of `protoc` * Created `buf.yaml` and `buf.gen.yaml` files in the two proto directories to control how `buf` generates/lints proto code. * Invoke `buf` instead of `protoc` * Added a `proto-format` make target. * Committed the reformatted proto files. * Added a `proto-lint` make target. * Integrated proto linting with CI * Fixed tons of proto linter warnings. * Got rid of deprecated builtin protoc-gen-go grpc plugin usage. Moved to direct usage of protoc-gen-go-grpc. * Unified all proto directories / go packages around using pb prefixes but ensuring all proto packages do not have the prefix.
24 lines
729 B
YAML
24 lines
729 B
YAML
version: v1
|
|
lint:
|
|
use:
|
|
- DEFAULT
|
|
allow_comment_ignores: true
|
|
except:
|
|
# we want to enable our Go packages to have a pb prefix to make goimports more
|
|
# intelligently handle fixing up imports and hopefully getting it right.
|
|
- PACKAGE_DIRECTORY_MATCH
|
|
# for internal protos we don't think we need proto versioning suffix for now
|
|
# also it would break the subscribe service if we change the proto package
|
|
- PACKAGE_VERSION_SUFFIX
|
|
|
|
# TODO - we should be able to remove these once we address the warnings
|
|
- FIELD_LOWER_SNAKE_CASE
|
|
- ENUM_VALUE_UPPER_SNAKE_CASE
|
|
- ENUM_ZERO_VALUE_SUFFIX
|
|
- ONEOF_LOWER_SNAKE_CASE
|
|
- ENUM_VALUE_PREFIX
|
|
service_suffix: ""
|
|
breaking:
|
|
use:
|
|
- FILE
|