Skip to content

Conversation

@azaroth42
Copy link
Collaborator

  • Change status to PR
  • Remove CR exit criteria (as already exited!)
  • Remove reference to exit criteria
  • Remove At Risk for AS2
  • Remove redundant string body appendix
@iherman
Copy link
Member

iherman commented Dec 21, 2016

I would not have removed the exit criteria... yes, it is not really relevant, but it is part of the documents' history, just like the changes' section. Would it be possible to put them back?

@iherman
Copy link
Member

iherman commented Dec 21, 2016

As far as I can see, the changes' sections were not updated to record the changes between the latest CR and the PR

@azaroth42
Copy link
Collaborator Author

I looked at other Proposed Recs (eg dwbp) and they didn't have exit criteria, so I took them out. I can put them back (just cut/paste), but I assume that for TR we'll take them out again, and compress the change history to only be differences with the CG's version?

There shouldn't be a reference to them in the status section though? Just the appendix?

@iherman
Copy link
Member

iherman commented Dec 21, 2016 via email

@azaroth42
Copy link
Collaborator Author

Deal :) My brief survey looking back through the announcements was 3 PRs without CR exit criteria, and one (webmention) with ... but webmention had it still in the Status ... so I think they're going to have to fix that for Rec!

Closing this and will re-edit rather than replace the text.

@azaroth42 azaroth42 closed this Dec 21, 2016
@azaroth42 azaroth42 deleted the pr-fixes branch December 21, 2016 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants