Add options to doc response

[gilwong00]

Part 2 of #2752.

Adding FieldLiteral to Message, Enum, Service, Method and EnumValue responses

[bufdev]

It's weird that we have message_options and then options where options is defined as "all options on the message". Do you instead want to deprecate message_options and say that this supersedes it:?

[gilwong00]

Yea we can do that. Should we also do the same with enum_options , method_options and service_options as well?

[jhump]

Yes. In all cases, the new repeated FieldLiteral options field deprecates the old *_options field.

[bufdev]

Perfect

[gilwong00]

Added [deprecated = true] to all *_options fields

[gilwong00]

I can completely remove *_options fields from the messages in a follow up PR after the changes in core ships.

[bufdev]

Can you move this to the bottom of the message definition, and add a comment "Use options instead."

  ...
  repeated FieldLiteral options = 10;
  // Use options instead.
  ServiceOptions service_options = 8;
}

Also, is ServiceOptions used elsewhere, or is this the only use? If not used elsewhere, also deprecate it:

message ServiceOptions {
  option deprecated = true;
  ...
}

Apply this across all of the below. Sorry for the back and forth.