REPEATED_FIELD_NAMES_PLURALIZED should account for uncountable names

[CoRfr]

While adding a field

repeated AdditionalInfo additional_info = 6;

protolint raised the following issue:

Repeated field name "additional_info" must be pluralized name "additional_infos"

My understanding is that info is uncountable, and therefore that the pluralized version of info is also info.

[yoheimuta]

That's right. Thank you for letting me know.

[yoheimuta]

📝 You can implement a workaround to adjust uncountable names by using a config file, such as

protolint/_example/config/.protolint.yaml

Lines 118 to 120 in 71e6503

	repeated_field_names_pluralized:
	uncountable_rules:
	- paper

.

[jduan-highnote]

Our CI job picked up the latest release (0.53.0) and now it's failing due to the following:

[protos/proto-onboarding/src/main/java/proto/onboarding/onboarding.proto:413:3] Repeated field name "preliminary_business_infos" must be pluralized name "preliminary_business_info"

How do I allow "preliminary_business_infos" in .protolintrc? Thanks!

[yoheimuta]

@jduan-highnote You can add plural_rules like the below.

    repeated_field_names_pluralized:
      plural_rules:
        "(?i)info$": "infos"

[jduan-highnote]

Tried our suggestion and it worked well! Thanks.