Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Editorial suggestions for Overview section #17

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

daidoji
Copy link

@daidoji daidoji commented Apr 3, 2024

No description provided.

@wenjing
Copy link
Contributor

wenjing commented May 7, 2024

Line 50: ", including additional security considerations" - emphasizes that having "additional" security considerations in the transport is one out of many considerations to have. Replacing that with "such as security considerations" may be misinterpreted as requiring or mainly about security considerations. If your edit is mainly about grammatical preference, then I would suggest "such as additional security considerations and others"? TSP does not require you having a secure transport. This point is much more important than grammatical niceties.

The rest looks ok.

@wenjing
Copy link
Contributor

wenjing commented May 7, 2024

Line 42: one more issue: a new term "trust-based ecosystem" is introduced but I think we should avoid. It's a term of complex meanings that we do not wish to engage with in TSP. It's a good concept in elsewhere, but introducing it here limits TSP and reduces its scope.

@daidoji
Copy link
Author

daidoji commented May 8, 2024

Roger, will make first edit and remove the second.

…ations aren't the only ones that the sender has to make
@daidoji
Copy link
Author

daidoji commented May 8, 2024

@wenjing changed per your comments.

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