Dear Jim

Many thanks for doing a thorough job in finding where changes are needed.

* In history.adoc, I think this issue number (139) should be mentioned - I 
believe we've agreed that this is needed to make the changes traceable, as for 
trac tickets. Since the issue mentions the pull request number, we don't need 
to put that in the history as well.

* In ch07.adoc, in the paragraph where you've made a small change, I notice 
there are some backticks ` which shouldn't be there. They are markup for 
verbatim text, as in Trac. Maybe this syntax is wrong for asciidoc? It's 
nothing to do with your change, but it could be fixed as well.

* In ch02.adoc, it could be helpful to include a CDL snippet showing a string 
array represented in the two different ways. You have changed all the examples 
to strings. That implies that you're thinking we should recommend the string 
type - yes? If so, we should say so. That also have to appears as a 
recommendation in the conformance document, and warnings would be produced for 
all use of char arrays in this way. Or are they equally acceptable? If so, we 
should say that.

Best wishes

Jonathan

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/cf-convention/cf-conventions/issues/139#issuecomment-407044230

Reply via email to