search

ietf-wg-cats / CATS-framework

Other
1 stars 4 forks source link

Update draft-ietf-cats-framework.md #91

Closed muzixing closed 2 months ago

muzixing commented 2 months ago

update to address Zongpeng's comments

muzixing commented 2 months ago

This update is agreed by Cheng and Zongpeng, other authors have not replied yet. Med, if you are ok with the update, please merge it.

muzixing commented 2 months ago

For the rest comments，my proposal is to go with two ways:

  1. keep it now, and send an email to the list to discuss the text of definition. change the definition in two drafts next time.
  2. provide new text, update it and ask for comment. Update it again when we get conclusion on the definition.

It will be better to update this revision ASAP, so that we can work on another revision to address rest issues.

boucadair commented 2 months ago

I suggest we don't make proposed changes for the pending points (that is, revert to the OLD text). For each pending point, create a separate PR. That would ease merging and converging. Thanks.

muzixing commented 2 months ago

I suggest we don't make proposed changes for the pending points (that is, revert to the OLD text). For each pending point, create a separate PR. That would ease merging and converging. Thanks.

I fully agree. So let's do in this way.

BTW, I merged two PRs already, cause the conflicts on this. You may revert them and merge this firstly, or you can edit it manually. Sorry for that :)

muzixing commented 2 months ago

BTW, ,Med, I think you can revert back the modification back to the old text if do not agree, and merge the PR. We can submit this revision this week. We can continue the discussion and update accordingly, thanks!