Convert inline TODO to issues

Author: jasonmccreary

src/BlueprintCommand.php

@@ -39,14 +39,13 @@ public function __construct(Filesystem $files)
     /**
      * Execute the console command.
      *
-     * @param \App\DripEmailer $drip
      * @return mixed
      */
     public function handle()
     {
         $contents = $this->files->get($this->argument('draft'));
 
-        $blueprint = new Blueprint(); // TODO: resolve or make static for extensibility
+        $blueprint = new Blueprint();
 
         $blueprint->registerLexer(new \Blueprint\Lexers\ModelLexer());
 
@@ -81,4 +80,4 @@ protected function getOptions()
     {
         return [];
     }
-}
+}

src/Generators/FactoryGenerator.php

@@ -21,7 +21,6 @@ public function output(array $tree): array
     {
         $output = [];
 
-        // TODO: what if changing an existing model
         $stub = $this->files->get(STUBS_PATH . '/factory.stub');
 
         /** @var \Blueprint\Model $model */
@@ -114,7 +113,7 @@ protected function fakerData(string $name)
     protected function fakerDataType(string $type)
     {
         $fakeableTypes = [
-            'id' => 'randomDigitNotNull', // TODO: override with closure generator
+            'id' => 'randomDigitNotNull',
             'string' => 'word',
             'text' => 'text',
             'date' => 'date()',
@@ -133,4 +132,4 @@ protected function fakerDataType(string $type)
 
         return $fakeableTypes[$type] ?? null;
     }
-}
+}

src/Generators/MigrationGenerator.php

@@ -22,7 +22,6 @@ public function output(array $tree): array
     {
         $output = [];
 
-        // TODO: what if changing an existing model
         $stub = $this->files->get(STUBS_PATH . '/migration.stub');
 
         /** @var \Blueprint\Model $model */
@@ -100,4 +99,4 @@ protected function getPath(Model $model)
     {
         return 'database/migrations/' . \Carbon\Carbon::now()->format('Y_m_d_His') . '_create_' . $model->tableName() . '_table.php';
     }
-}
+}

src/Generators/ModelGenerator.php

@@ -20,7 +20,6 @@ public function output(array $tree): array
     {
         $output = [];
 
-        // TODO: what if changing an existing model
         $stub = $this->files->get(STUBS_PATH . '/model/class.stub');
 
         /** @var \Blueprint\Model $model */
@@ -149,4 +148,4 @@ private function propertyStub(string $stub)
 
         return $stubs[$stub];
     }
-}
+}

tests/Feature/Generator/MigrationGeneratorTest.php

@@ -73,7 +73,6 @@ public function modelTreeDataProvider()
             ['definitions/readme-example.bp', 'database/migrations/timestamp_create_posts_table.php', 'migrations/readme-example.php'],
             ['definitions/model-identities.bp', 'database/migrations/timestamp_create_relationships_table.php', 'migrations/identity-columns.php'],
             ['definitions/model-modifiers.bp', 'database/migrations/timestamp_create_modifiers_table.php', 'migrations/modifiers.php'],
-            // TODO: optimizations like nullableTimestamp, unsignedInteger, etc
         ];
     }
-}
+}

tests/Feature/Generator/ModelGeneratorTest.php

@@ -77,7 +77,6 @@ public function modelTreeDataProvider()
     {
         return [
             ['definitions/readme-example.bp', 'app/Post.php', 'models/readme-example.php'],
-            // TODO: relationships
         ];
     }
-}
+}