From d6b210ea534b3044b48ba92eac18ae8f2b803490 Mon Sep 17 00:00:00 2001 From: Martin Dosch Date: Sun, 1 Nov 2020 12:28:26 +0100 Subject: [PATCH 1/3] Add tag [Needs Editor Action] for non-editorial changes to deferred XEPs. --- docs/TRIAGING.md | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/docs/TRIAGING.md b/docs/TRIAGING.md index dbe82654..68b07b9c 100644 --- a/docs/TRIAGING.md +++ b/docs/TRIAGING.md @@ -138,7 +138,8 @@ If the PR is not touching a XEP, this guide does not apply. 4. Stop. 4. If the XEP is in Deferred state and the changes are not purely editorial, - add a note to move the XEP to Experimental state. + add a note to move the XEP to Experimental state and mark the PR as + [Needs Editor Action]. 5. Otherwise, mark the PR as [Ready to Merge], linking the XEP Author’s approval for documentation purposes. From b9160265f4a3239619cd79832d684c7f387b6190 Mon Sep 17 00:00:00 2001 From: Martin Dosch Date: Sun, 1 Nov 2020 12:33:27 +0100 Subject: [PATCH 2/3] Change no longer appropriate wording. --- docs/TRIAGING.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/TRIAGING.md b/docs/TRIAGING.md index 68b07b9c..5715b182 100644 --- a/docs/TRIAGING.md +++ b/docs/TRIAGING.md @@ -140,8 +140,8 @@ If the PR is not touching a XEP, this guide does not apply. 4. If the XEP is in Deferred state and the changes are not purely editorial, add a note to move the XEP to Experimental state and mark the PR as [Needs Editor Action]. - 5. Otherwise, mark the PR as [Ready to Merge], linking the XEP Author’s - approval for documentation purposes. + 5. Mark the PR as [Ready to Merge], linking the XEP Author’s approval for + documentation purposes. 6. Mark the PR as [Ready to Merge]. From 66fb96fd1733e28941526c413eed8bb7c9e3a4a6 Mon Sep 17 00:00:00 2001 From: Martin Dosch Date: Sun, 1 Nov 2020 12:42:43 +0100 Subject: [PATCH 3/3] Change order of steps for Experimental/Deferred XEPs. --- docs/TRIAGING.md | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/docs/TRIAGING.md b/docs/TRIAGING.md index 5715b182..d7438a08 100644 --- a/docs/TRIAGING.md +++ b/docs/TRIAGING.md @@ -125,7 +125,10 @@ If the PR is not touching a XEP, this guide does not apply. 2. If the PR adds a revision block and does not bump the minor-level (second) version number part, add the [Needs Version Block] label and a comment explaining the situation for the Processing Editor. - 3. If the issue has not been discussed on the standards list *or* if + 3. If the XEP is in Deferred state and the changes are not purely editorial, + add a note to move the XEP to Experimental state and mark the PR as + [Needs Editor Action]. + 4. If the issue has not been discussed on the standards list *or* if the XEP Author has not been involved in the discussion *or* the XEP Author has not explicitly ACKed the PR: @@ -137,11 +140,8 @@ If the PR is not touching a XEP, this guide does not apply. for each author in either the XEP or in xep.ent. 4. Stop. - 4. If the XEP is in Deferred state and the changes are not purely editorial, - add a note to move the XEP to Experimental state and mark the PR as - [Needs Editor Action]. - 5. Mark the PR as [Ready to Merge], linking the XEP Author’s approval for - documentation purposes. + 5. Otherwise, mark the PR as [Ready to Merge], linking the XEP Author’s + approval for documentation purposes. 6. Mark the PR as [Ready to Merge].