Problem/Motivation

This module uses the Robo dependency which can lead to problems when you don't already have Robo installed. Robo is also only usable in combination with Drush and will not work if you use the interface for exporting.
Robo also isn't required in the composer.json.

Proposed resolution

Remove the Robo dependency and use the default Drupal logger.

Related issues

Robo\Robo not found

CommentFileSizeAuthor
#2 Remove_Robo_dependency-3020962-2.patch1.65 KBJasperh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jasperh created an issue. See original summary.

Jasperh’s picture

Jasperh CreditAttribution: Jasperh at LimoenGroen commented
Status: Active » Needs review
FileSize
Remove_Robo_dependency-3020962-2.patch1.65 KB

This patch removes the Robo dependency and replaces it with the default Drupal logger.

Sutharsan’s picture

Sutharsan CreditAttribution: Sutharsan at LimoenGroen commented
Status: Needs review » Reviewed & tested by the community

Makes sense to remove this dependency. Patch looks good.

martin_klima credited mirom.

martin_klima’s picture

martin_klima
he/his
Primary language Czech
CreditAttribution: martin_klima at Czech Drupal Association commented
Issue tags: +DevDaysTransylvania

Thank you for the patch. I improved your idea and used DI. Thanks, Mirom for helping hands.

martin_klima’s picture

martin_klima
he/his
Primary language Czech
CreditAttribution: martin_klima at Czech Drupal Association commented
Status: Reviewed & tested by the community » Fixed

  • martin_klima committed f4e90ce on 8.x-1.x authored by Jasperh 
    Issue #3020962 by Jasperh, Sutharsan, mirom, martin_klima: Remove Robo...

Status: Fixed » Closed (fixed)

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