#2625696: Make migrations themselves plugins instead of config entities removed the migration template concept, but there's still a test module (now unused) for it. Let's remove it.

CommentFileSizeAuthor
#2 remove_unused_migration-2748553-2.patch2.13 KBmikeryan

Comments

mikeryan created an issue. See original summary.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Status: Active » Needs review
StatusFileSize
new remove_unused_migration-2748553-2.patch2.13 KB
oadaeh’s picture

oadaeh commented

Shouldn't this be being applied to 8.2.x first?

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented

@oadeah - The policy is to file bug reports against 8.1.x and other issues against 8.2.x - the core committer will apply the bug report patch to both 8.1.x and 8.2.x.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented

@oadeah: See https://www.drupal.org/core/backport-policy.

oadaeh’s picture

oadaeh commented

@mikeryan: thanks for the pointer. I missed that change in policy.

oadaeh’s picture

oadaeh commented
Status: Needs review » Reviewed & tested by the community

I hate to just let this sit and wait, since I already checked it.
The patch applies cleanly to both 8.1.x and 8.2.x, and it does what it says it does.

  • catch committed dd70b2f on 8.2.x 
    Issue #2748553 by mikeryan: Remove unused migration template_test module
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.2.x and cherry-picked to 8.1.x. Thanks!

Status: Fixed » Closed (fixed)

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