stoffi92 / rfc5575bis

0 stars 0 forks source link

IESG Warren Kumari: removed all references to a opaque-key #217

Closed stoffi92 closed 4 years ago

stoffi92 commented 4 years ago

“ This specification has removed all references to a opaque-key property. BGP is able to understand the NLRI encoding.” 2 nits: 1: a opaque-key -> an opaque-key (or the opaque-key) 2: BGP is able to... seems a bit odd - BGP is just a protocol, it doesn’t really understand anything. I think that a better option would be best “BGP implementations are”. Again this is just a nit, feel free to address it or not...

suehares commented 4 years ago

the change from /a opaque-key/ to /an opaque-key/ - is good /BGP is able/ to /BGP implementations are/ is good

suehares commented 4 years ago

Do you want me to leave this open until you've changed the text?

stoffi92 commented 4 years ago

//doc /a opaque-key/ to /an opaque-key/ /BGP is able/ to /BGP implementations are/