Discovered via #340723: Make modules and installation profiles only require .info.yml files

This slows down the installer and tests.

CommentFileSizeAuthor
#2 drupal8.theme-maintenance-dupe.2.patch647 bytessun
drupal8.theme-maintenance-dupe.0.patch605 bytessun

Comments

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Needs review » Reviewed & tested by the community

Tested installer - everything looks fine

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
StatusFileSize
new drupal8.theme-maintenance-dupe.2.patch647 bytes

Actually, _theme_load_offline_registry() was removed as part of #1886448: Rewrite the theme registry into a proper service

Thus, this was caused by a faulty re-roll / merge conflict resolution of the patch over there, and it's actually the other way around.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2: drupal8.theme-maintenance-dupe.2.patch, failed testing.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs work » Needs review

2: drupal8.theme-maintenance-dupe.2.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 2: drupal8.theme-maintenance-dupe.2.patch, failed testing.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Needs work » Needs review

2: drupal8.theme-maintenance-dupe.2.patch queued for re-testing.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Probably my fault.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Nice catch.

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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