Skip to content

Conversation

noahdietz
Copy link
Collaborator

It is reasonable to allow the use of a name field in a nested message since nested messages are most often authored as a small piece of a larger message, thus never really candidates for being a resource.

@noahdietz noahdietz requested a review from a team as a code owner January 24, 2024 00:16
Copy link
Contributor

@andrei-scripniciuc andrei-scripniciuc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for sending this! could you update the rule docs as well with this exception?

@noahdietz
Copy link
Collaborator Author

thanks for sending this! could you update the rule docs as well with this exception?

Yep! Done.

@noahdietz noahdietz merged commit 16316a5 into googleapis:main Jan 25, 2024
@noahdietz noahdietz deleted the nested-name branch January 25, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants