Open Bug 1402533 Opened 2 years ago Updated 2 years ago

Look for commits that change .pb.h or files without also changing a .proto file


(Testing :: General, enhancement, P3)

Version 3


(Not tracked)


(Reporter: francois, Unassigned)



Files that end in .pb.h and are generated from a protobuf definition file (.proto) and start with this warning:

// Generated by the protocol buffer compiler.  DO NOT EDIT!
// source: LayerScopePacket.proto

However, it can slip through reviews and lead to commits where the .proto is out of sync with the generated files and cannot re-generate the files as they are in the codebase.

Here are two examples of commits which forgot to update the .proto:

It would be great if we could detect that automatically and fail the build on Try.
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.