Re-revised PR for CR2 - Githubissues

w3c / web-annotation

Web Annotation Working Group repository, see README for links to specs

https://w3c.github.io/web-annotation/

Other

141 stars 30 forks source link

Re-revised PR for CR2 #379

Closed azaroth42 closed 7 years ago

iherman commented 7 years ago

Great. @azaroth42, some minor issues before merging:

The vocab document still has an open issue listed in the status section. Isn't it possible to remove that, too? Wasn't that an "at risk" feature that has been removed?
It may be worth emphasizing in the changes' section that some feature were removed because they were at risk, and some were removed because they did not get through the original exit criteria. Just to be "elegant":-)
Before merging, can you make a copy of the files in the /TR directory in admin? That is the reference we would use for the checker and the CR transition request...

Thanks!

iherman commented 7 years ago

@azaroth42 @tcole3 :

I have prepared two drafts for the transition mailings as part of this PR:

admin/CRTransitionAdmin/3rd CRTransitionRequest.md
admin/CRTransitionAdmin/3rd PublicationRequest.md

I have already sent a question to the director whether a simplified transition (see the first mail) is enough for this case; I hope it is. Please review it and change any details that you see fit.

The mail should go out on Monday 14th. @azaroth42 if the files do not pass pubrules' checker on that day, that is fine, it can be done on, say, Tuesday; the content should be final, though.

iherman commented 7 years ago

@azaroth42 : I took care of my own comments. (I realized that the at-risk in the vocab is related to the the AS vocabulary, so it remains as is for now.) I have also checked the files with pubrules' checker and the link checker, and took care of a link error.

Will merge now.