diff options
author | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-05-12 09:32:52 +0300 |
---|---|---|
committer | Patrick Steinhardt <psteinhardt@gitlab.com> | 2022-05-12 09:32:52 +0300 |
commit | 42fab8fc526215f9426bc9f459f9e6da0951c574 (patch) | |
tree | 94a6ca646e9b59bc5a47bd0994147797925855b9 | |
parent | d4df39f4cfa395cd2a8f72a52e08495029a2d1d4 (diff) | |
parent | 8e3ad7f93edb20ad5ecb9024399c8044f7f19bbe (diff) |
Merge branch 'wc-proto-contrib' into 'master'
doc: Update Protobuf verification docs
See merge request gitlab-org/gitaly!4551
-rw-r--r-- | doc/protobuf.md | 19 |
1 files changed, 7 insertions, 12 deletions
diff --git a/doc/protobuf.md b/doc/protobuf.md index f7e188323..4ed62296f 100644 --- a/doc/protobuf.md +++ b/doc/protobuf.md @@ -358,18 +358,6 @@ declared. This is done via the `go_package` option: This allows other protobuf files to locate and import the Go generated stubs. -## Contributing - -The CI at https://gitlab.com/gitlab-org/gitaly-proto regenerates the -client libraries to guard against the mistake of updating the .proto -files but not the client libraries. This check uses `git diff` to look -for changes. Some of the code in the Go client libraries is sensitive -to implementation details of the Go standard library (specifically, -the output of gzip). **Use the same Go version as .gitlab-ci.yml (Go -1.13)** when generating new client libraries for a merge request. - -[DCO + License](CONTRIBUTING.md) - ## Workflows ### Generating Protobuf sources @@ -382,6 +370,13 @@ libraries before committing your change. make proto ``` +### Verifying Protobuf definitions + +Gitaly provides a `make lint-proto` target to verify that Protobuf definitions +conform to our coding style. Furthermore, Gitaly's CI verifies that sources +generated from the definitions are up-to-date by regenerating sources and then +running `no-proto-changes`. + ### Deprecating an RPC call See [PROCESS.md](PROCESS.md#rpc-deprecation-process). |