- Issue created by @geek-merlin
- π¦πΊAustralia dpi Perth, Australia
MR welcome. Thanks for the doc, I think thats critical to this issue.
Tests not required for this change.
- π¨π¦Canada ambient.impact Toronto
ambient.impact β made their first commit to this issueβs fork.
- π¨π¦Canada ambient.impact Toronto
ambient.impact β changed the visibility of the branch 3472641-wrong-module-name-limit to hidden.
- Merge request !44Issue #3472641: Use Drupal core's module name limit in HuxCompilerPass. β (Merged) created by ambient.impact
- Merge request !45Issue #3472641: Use Drupal core's module name limit in HuxCompilerPass. β (Closed) created by ambient.impact
- Merge request !46Issue #3472641: Use Drupal core's module name limit in HuxCompilerPass. β (Closed) created by ambient.impact
- π¨π¦Canada ambient.impact Toronto
I created three merge requests, one for each of the currently supported branches. Not sure if there was a better way to do that, but there ya go. Also used Drupal core's constant that they added as part of that change record so we don't even have to know what the limit is, so if they ever decide to change it, Hux just uses the new limit.
- π¦πΊAustralia dpi Perth, Australia
All good, normally I'd cherry pick across branches I deem supported.
I've made minor changes to satisfy linting to one branch. So I'll do a quick cherry pick in this case.
-
dpi β
committed 143a855a on 1.8.x authored by
ambient.impact β
Issue #3472641 by ambient.impact, geek-merlin: π Use Drupal core's...
-
dpi β
committed 143a855a on 1.8.x authored by
ambient.impact β
-
dpi β
committed a9fdbb82 on 1.7.x authored by
ambient.impact β
Issue #3472641 by ambient.impact, geek-merlin: π Use Drupal core's...
-
dpi β
committed a9fdbb82 on 1.7.x authored by
ambient.impact β
Automatically closed - issue fixed for 2 weeks with no activity.