Problem/Motivation

After #3583903: Move the @defgroup comments into .api.php files and #3584468: Convert the existing hooks into OOP hooks are closed, the .module files will be no longer needed.

Proposed resolution

Remove the existing .module files.

Issue fork examples-3584471

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

avpaderno created an issue. See original summary.

oily made their first commit to this issue’s fork.

oily’s picture

oily commented

edited

oily opened merge request !119

oily’s picture

oily commented
Status: Active » Needs review

avpaderno changed the visibility of the branch 3584471-remove-the-.module to hidden.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Assigned: Unassigned » avpaderno

oily opened merge request !120

oily’s picture

oily commented

Not sure about the examples.module? I forgot about it when converting to OOP hooks in issue 3584468..

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Issue summary: View changes
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Assigned: avpaderno » Unassigned
Status: Needs review » Reviewed & tested by the community

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented

I left the main module for #3585588: Change how links for sub-modules are shown, to avoid changing hook implementations twice.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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