- Issue created by @amstercad
- @amstercad opened merge request.
- Issue was unassigned.
Hey there, thanks for the MR, I wonder if the dependencies should be like
sdc:sdc
or not?
If so could you maybe update your MR? thanks- 🇺🇸United States amstercad
You make a really good point!
I tested it as written, and followed this documentation example → .
I missed the nuance you understood right away until you brought it to my attention.
That said, shouldn't it be:
- drupal:sdc
- drupal:serialization...since sdc & serialization is in Drupal core? I'll test once I have a moment for the task.
- Status changed to Needs review
over 1 year ago 7:17am 20 October 2023 - @amstercad opened merge request.
@Amstercard closing this ticket, feel free to re-open a new one if needed
- 🇺🇸United States amstercad
Sorry for all the confusion trying to submit a merge-able patch. This GitLab bug was tripping me up.
-
doxigo →
committed 450da2c7 on 6.0.x authored by
Amstercad →
Issue #3393112: .info must include module dependencies
-
doxigo →
committed 450da2c7 on 6.0.x authored by
Amstercad →
- Status changed to Fixed
about 1 year ago 8:34am 19 November 2023 Great job getting the MR updated and thanks for your contribution Lee, this is merged
Automatically closed - issue fixed for 2 weeks with no activity.