Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This is the list of draft change notices: https://www.drupal.org/list-changes/drafts
There are some really really old change notices in that list. They should probably mostly either be published (if valid) or deleted (if the issues were abandoned or they were temporarily created in order to add something to another change notice).
Change notices in "draft" state do not show up on the main List Changes page, so having notices in Draft status is not good.
I'm tempted to mark this "critical".
Committers: Are you publishing draft change notices when you commit issues that have them?
Comments
Comment #1
jhedstromI reviewed about half of these, publishing the ones that were obvious oversights.
Here's a list that needs deleting (I don't have the permissions)
https://www.drupal.org/node/1972428
https://www.drupal.org/node/2106757
https://www.drupal.org/node/2208411
https://www.drupal.org/node/2234253
https://www.drupal.org/node/2263009
And these are for 8.1.x, not sure what, if anything, needs to be done there (I've updated the CNs to indicate 8.1.x, but the issues may or may not be revisited at that point).
https://www.drupal.org/node/2197189
These are somewhat abandoned, and debatable for 8.0x
https://www.drupal.org/node/2208801
https://www.drupal.org/node/2216617
https://www.drupal.org/node/2228363
https://www.drupal.org/node/2305883
https://www.drupal.org/node/2316145
These link to no issue, and can probably be deleted, or need updating to link to the appropriate issue.
https://www.drupal.org/node/2218621
https://www.drupal.org/node/2241645
https://www.drupal.org/node/2245853
https://www.drupal.org/node/2247473
https://www.drupal.org/node/2270461
https://www.drupal.org/node/2311213
Comment #2
jhodgdonThanks! I went through that list of "delete em":
https://www.drupal.org/node/2263009 -- I deleted that and redirected it to https://www.drupal.org/node/1272696 since the body said "don't use this, use that" anyway.
https://www.drupal.org/node/2208411 -- links to a "won't fix" issue, so deleted that without a redirect.
https://www.drupal.org/node/2106757 - agreed this should be deleted. It was a change that was later reversed, and there is not a change notice for the reversal.
https://www.drupal.org/node/1972428 I removed and redirected to https://www.drupal.org/node/2274223
But... https://www.drupal.org/node/2234253 leads to issue https://www.drupal.org/node/2228125 which is marked "closed/fixed" and there is not a replacement change notice on it. So I didn't do anything there. Not sure what the right thing to do is.
Comment #3
jacktonkin CreditAttribution: jacktonkin as a volunteer commentedThe issue linked to from https://www.drupal.org/node/2234253 was initially about renaming the
'base theme'
key in theme info.yml files to something without a space to get rid of the quotes. It turns out that YML (like D7 .info files) does't require keys with spaces to be quoted, so the patch that was committed in #2228125: 'base theme' property in theme .info.yml files does not need to be wrapped in quotes simply removed the quotes from core's themes.I don't think this clarification requires a change record, so the draft should be deleted.
Comment #4
jhodgdonAh, you're right. I've deleted that change notice then.
Comment #7
Wim LeersThis was fixed long ago.