Add support for drush code generator

Created on 4 September 2023, about 1 year ago
Updated 27 September 2023, about 1 year ago

Problem/Motivation

I would appreciate it if we could use drush code generator to generate the boilerplate for new services. I'd be happy to provide a basic implementation, which others could build on top of to make it more sophisticated, then.

Is that something the maintainers of this module would consider merging?

Feature request
Status

Fixed

Version

9.3

Component

Code

Created by

🇩🇪Germany jurgenhaas Gottmadingen

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @jurgenhaas
  • 🇮🇹Italy aronne

    Hi @jurgenhaas
    yes we'd gladly merge that feature if you're gonna open a MR

    Regards,
    aronne

  • Open on Drupal.org →
    Core: 10.0.7 + Environment: PHP 8.2 & MySQL 8
    last update about 1 year ago
    Waiting for branch to pass
  • @jurgenhaas opened merge request.
  • Status changed to Needs review about 1 year ago
  • 🇩🇪Germany jurgenhaas Gottmadingen

    I've added a first generator with templates which can be called with drush http_client_manager:service. The user gets asked a couple of questions and then a set of files is being created from the given answers.

    Note: this is built for the latest version of the code generator, so Drush version 12 is required in this case.

    This approach only covers a fraction of the possibilities that this amazing module provides, but it may be an easy starting point for new users. Of course, providing such a code generator generates some sort of default, which users are going to assume is THE way to use this module. I know, this can be opinionated and therefore, the review should really determine, if the output is what you want to deliver as a recommended way of usage.

    Whether we want to make the code generator more sophisticated or leave it on its intro level is totally up to the maintainers. For my own use case, where I use it for ECA TV live sessions to show how http client manager and ECA can work together. this is sufficient.

  • Open on Drupal.org →
    Core: 10.0.7 + Environment: PHP 8.2 & MySQL 8
    last update about 1 year ago
    Waiting for branch to pass
  • 🇮🇹Italy aronne

    I'm gonna upload the interdiff for the latest version.

  • Open on Drupal.org →
    Core: 10.0.7 + Environment: PHP 8.2 & MySQL 8
    last update about 1 year ago
    Waiting for branch to pass
  • 🇮🇹Italy aronne

    And this is the whole patch.

    • aronne authored 74e335ab on 9.3.x
      Issue #3385117 by jurgenhaas, aronne: Add support for drush code...
  • Status changed to Fixed about 1 year ago
  • 🇮🇹Italy aronne

    Thank you @jurgenhaas for your valuable contribution.
    Regards,
    aronne

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

Production build 0.71.5 2024