Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Replace "label" check with "cardinality" check #2927

Merged
merged 8 commits into from
May 2, 2024
Merged

Replace "label" check with "cardinality" check #2927

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
bc036af
replace FIELD_SAME_LABEL with FIELD_SAME_CARDINALITY; introduce wire-…
jhump Apr 29, 2024
2710aa3
update oneof checks to ignore synthetic oneofs; these are now handled…
jhump Apr 29, 2024
6b0d59d
add back test for FIELD_SAME_LABEL, to verify its config swaps in the…
jhump Apr 30, 2024
ecd898b
change v1 to match v2: different levels of cardinality check based on…
jhump Apr 30, 2024
8199ab3
update test expectations: if 'use' FIELD_SAME_LABEL directly, it can …
jhump Apr 30, 2024
0b89281
oops, missed a test
jhump May 1, 2024
48dcbac
Merge branch 'dev' into jh/new-buf-breaking-check
jhump May 1, 2024
abc325c
don't report errors in triplicate if FIELD_SAME_LABEL not explicitly …
jhump May 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 4 additions & 2 deletions private/buf/cmd/buf/buf_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -639,6 +639,7 @@ ENUM_SAME_TYPE FILE, PACKAGE
ENUM_VALUE_NO_DELETE FILE, PACKAGE Checks that enum values are not deleted from a given enum.
EXTENSION_MESSAGE_NO_DELETE FILE, PACKAGE Checks that extension ranges are not deleted from a given message.
FIELD_NO_DELETE FILE, PACKAGE Checks that fields are not deleted from a given message.
FIELD_SAME_CARDINALITY FILE, PACKAGE Checks that fields have the same cardinalities in a given message.
FIELD_SAME_CTYPE FILE, PACKAGE Checks that fields have the same value for the ctype option.
FIELD_SAME_JSTYPE FILE, PACKAGE Checks that fields have the same value for the jstype option.
FIELD_SAME_TYPE FILE, PACKAGE Checks that fields have the same types in a given message.
Expand Down Expand Up @@ -668,7 +669,6 @@ RPC_NO_DELETE FILE, PACKAGE
ENUM_VALUE_SAME_NAME FILE, PACKAGE, WIRE_JSON Checks that enum values have the same name.
FIELD_SAME_JSON_NAME FILE, PACKAGE, WIRE_JSON Checks that fields have the same value for the json_name option.
FIELD_SAME_NAME FILE, PACKAGE, WIRE_JSON Checks that fields have the same names in a given message.
FIELD_SAME_LABEL FILE, PACKAGE, WIRE_JSON, WIRE Checks that fields have the same labels in a given message.
FIELD_SAME_ONEOF FILE, PACKAGE, WIRE_JSON, WIRE Checks that fields have the same oneofs in a given message.
FILE_SAME_PACKAGE FILE, PACKAGE, WIRE_JSON, WIRE Checks that files have the same package.
MESSAGE_SAME_MESSAGE_SET_WIRE_FORMAT FILE, PACKAGE, WIRE_JSON, WIRE Checks that messages have the same value for the message_set_wire_format option.
Expand All @@ -686,9 +686,11 @@ PACKAGE_NO_DELETE PACKAGE
PACKAGE_SERVICE_NO_DELETE PACKAGE Checks that services are not deleted from a given package.
ENUM_VALUE_NO_DELETE_UNLESS_NAME_RESERVED WIRE_JSON Checks that enum values are not deleted from a given enum unless the name is reserved.
FIELD_NO_DELETE_UNLESS_NAME_RESERVED WIRE_JSON Checks that fields are not deleted from a given message unless the name is reserved.
FIELD_WIRE_JSON_COMPATIBLE_CARDINALITY WIRE_JSON Checks that fields have wire and JSON compatible cardinalities in a given message.
FIELD_WIRE_JSON_COMPATIBLE_TYPE WIRE_JSON Checks that fields have wire and JSON compatible types in a given message.
ENUM_VALUE_NO_DELETE_UNLESS_NUMBER_RESERVED WIRE_JSON, WIRE Checks that enum values are not deleted from a given enum unless the number is reserved.
FIELD_NO_DELETE_UNLESS_NUMBER_RESERVED WIRE_JSON, WIRE Checks that fields are not deleted from a given message unless the number is reserved.
FIELD_WIRE_COMPATIBLE_CARDINALITY WIRE Checks that fields have wire-compatible cardinalities in a given message.
FIELD_WIRE_COMPATIBLE_TYPE WIRE Checks that fields have wire-compatible types in a given message.
`
testRunStdout(
Expand Down Expand Up @@ -782,7 +784,7 @@ RPC_NO_DELETE FILE, PACKAGE
ENUM_VALUE_SAME_NAME FILE, PACKAGE, WIRE_JSON Checks that enum values have the same name.
FIELD_SAME_JSON_NAME FILE, PACKAGE, WIRE_JSON Checks that fields have the same value for the json_name option.
FIELD_SAME_NAME FILE, PACKAGE, WIRE_JSON Checks that fields have the same names in a given message.
FIELD_SAME_LABEL FILE, PACKAGE, WIRE_JSON, WIRE Checks that fields have the same labels in a given message.
FIELD_SAME_CARDINALITY FILE, PACKAGE, WIRE_JSON, WIRE Checks that fields have the same cardinalities in a given message.
FIELD_SAME_ONEOF FILE, PACKAGE, WIRE_JSON, WIRE Checks that fields have the same oneofs in a given message.
FIELD_SAME_TYPE FILE, PACKAGE, WIRE_JSON, WIRE Checks that fields have the same types in a given message.
MESSAGE_SAME_MESSAGE_SET_WIRE_FORMAT FILE, PACKAGE, WIRE_JSON, WIRE Checks that messages have the same value for the message_set_wire_format option.
Expand Down