Problem/Motivation

#2991207: Drupal core should inform the user of the security coverage for the site's installed minor version including final 8.x LTS releases was committed to 9.0.x and 8.9.x with the following code comment:

   * @todo In https://www.drupal.org/node/2608062 determine how we will know
   *    what the final minor release of a particular major version will be. This
   *    method should not return a version beyond that minor.

#2608062: [META] Requirements for tagging Drupal 9.0.0-alpha1 is the meta for tracking 9.0.0-alpha1 requirements, and is now closed (fixed).

As @Gábor Hojtsy pointed out at #2991207-249: Drupal core should inform the user of the security coverage for the site's installed minor version including final 8.x LTS releases.3, we need a better place to point this @todo.

Proposed resolution

Fix the comment so it points to #2998285: Add information on later releases to updates.drupal.org instead.

Remaining tasks

  1. Find or create the right issue for this.
  2. Update the comment.
  3. Review
  4. RTBC
  5. Commit and cherry-pick to all of 9.0.x, 8.9.x and 8.8.x branches

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

None.

CommentFileSizeAuthor
#4 3117188-4.patch704 bytesdww
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dww created an issue. See original summary.

dww credited Gábor Hojtsy.

dww’s picture

Issue summary: View changes

Crediting @Gábor for noticing this...

dww’s picture

Issue summary: View changes
Status: Active » Needs review
FileSize
704 bytes
tedbow’s picture

Status: Needs review » Reviewed & tested by the community

Yep this look like the right issue

alexpott’s picture

Version: 9.0.x-dev » 8.9.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed 951ca46b8c to 9.0.x and fa68fad460 to 8.9.x. Thanks!

  • alexpott committed 951ca46 on 9.0.x
    Issue #3117188 by dww, Gábor Hojtsy: Change @todo comment in core/...

  • alexpott committed fa68fad on 8.9.x
    Issue #3117188 by dww, Gábor Hojtsy: Change @todo comment in core/...
dww’s picture

Version: 8.9.x-dev » 8.8.x-dev
Issue summary: View changes
Status: Fixed » Reviewed & tested by the community

FYI, the thing this is fixing was backported all the way to 8.8.x, so the faulty comment lives there, too. We can haz backport pleaz? ;)

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Backported to 8.8.x as a docs only fix.

  • alexpott committed e90d105 on 8.8.x
    Issue #3117188 by dww, Gábor Hojtsy: Change @todo comment in core/...
dww’s picture

Issue summary: View changes

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.