Remove all code. Leave:

  • info file
  • composer.json
  • initial routing file
  • initial drush command file and command itself
  • Initial README.module
CommentFileSizeAuthor
#6 cyr2lat-cleanup-3028588-6.patch35.63 KBmikispeed
#2 cyr2lat-cleanup-3028588-1.patch35.63 KBmikispeed

Comments

mikispeed created an issue. See original summary.

mikispeed’s picture

mikispeed commented
StatusFileSize
new cyr2lat-cleanup-3028588-1.patch35.63 KB

Initial cleanup of the module patch.

botanic_spark’s picture

botanic_spark commented 
  1. +++ b/composer.json
@@ -1,6 +1,6 @@
+  "description": "Cyrillic to Latin Custom Transliteration module.",

    Remove word "Custom" 

  2. +++ b/cyr2lat.info.yml
@@ -1,5 +1,9 @@
+description: 'Cyrillic to Latin Custom Transliteration module.'

    Remove word "Custom"

Otherwise, looks ok.

botanic_spark’s picture

botanic_spark commented
Status: Active » Needs work
botanic_spark’s picture

botanic_spark commented 
+++ b/src/Commands/Cyr2LatCommands.php
@@ -22,148 +20,7 @@ class Cyr2LatCommands extends DrushCommands {
+    drush_print("Dummy command.");

The function is deprecated. Replace with recommended one.

mikispeed’s picture

mikispeed commented
StatusFileSize
new cyr2lat-cleanup-3028588-6.patch35.63 KB

Updated per comments.

mikispeed’s picture

mikispeed commented
Status: Needs work » Needs review
botanic_spark’s picture

botanic_spark commented

Looks good.

botanic_spark’s picture

botanic_spark commented
Status: Needs review » Reviewed & tested by the community

  • mikispeed committed ecc15bb on 8.x-1.x 
    Issue #3028588 by mikispeed: Cleanup initial D8 module version
mikispeed’s picture

mikispeed commented
Status: Reviewed & tested by the community » Fixed

Issue #3028588 by mikispeed, botanic_spark: Cleanup initial D8 module version

Status: Fixed » Closed (fixed)

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