Followup to PR https://github.com/md-systems/ultimate_cron/pull/34

CommentFileSizeAuthor
composer-add-ultimate-cron.patch587 bytesnaveenvalecha
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

naveenvalecha created an issue. See original summary.

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer and at Acquia for Acquia commented

Please do give credit to user https://www.drupal.org/u/mscharley

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer and at Acquia for Acquia commented
Assigned: naveenvalecha » Unassigned
Status: Needs review » Reviewed & tested by the community

As the code was written by mscharley and the patch looks good to me. So doing RTBC :)

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Parent issue: » #2660208: Plan for Ultimate Cron 8.x-2.0-beta1
naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer commented
Status: Reviewed & tested by the community » Closed (won't fix)

Closing this issue as this is not required to add composer.json if the module does not require any additional PHP library