#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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikeryan created an issue. See original summary.

mikeryan’s picture

Status: Active » Needs review
FileSize
2.13 KB
oadaeh’s picture

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

mikeryan’s picture

@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

oadaeh’s picture

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

oadaeh’s picture

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

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.