@@ -407,6 +407,7 @@ def add_rule(
407407
408408class MessageOneofRule (Rules ):
409409 """Validates a single buf.validate.MessageOneofRule given via the message option (buf.validate.message).oneof"""
410+
410411 def __init__ (self , fields : typing .List [descriptor .FieldDescriptor ], required : bool ):
411412 self ._fields = fields
412413 self ._required = required
@@ -417,14 +418,14 @@ def validate(self, ctx: RuleContext, msg: message.Message):
417418 ctx .add (
418419 Violation (
419420 rule_id = "message.oneof" ,
420- message = f"only one of { ", " .join ([field .name for field in self ._fields ])} can be set" ,
421+ message = f"only one of { ', ' .join ([field .name for field in self ._fields ])} can be set" ,
421422 )
422423 )
423424 if self ._required and num_set_fields == 0 :
424425 ctx .add (
425426 Violation (
426427 rule_id = "message.oneof" ,
427- message = f"one of { ", " .join ([field .name for field in self ._fields ])} must be set" ,
428+ message = f"one of { ', ' .join ([field .name for field in self ._fields ])} must be set" ,
428429 )
429430 )
430431
@@ -457,7 +458,7 @@ def add_oneof(
457458 if name in self ._desc .fields_by_name :
458459 fields .append (self ._desc .fields_by_name [name ])
459460 else :
460- msg = f" field \ "{ name } \ " not found in message { self ._desc .full_name } "
461+ msg = f' field "{ name } " not found in message { self ._desc .full_name } '
461462 raise CompilationError (msg )
462463 self ._oneofs .append (MessageOneofRule (fields , rule .required ))
463464
0 commit comments