Merge pull request #1 from Hi-Folks/feat/rector-php

Adding Rector PHP

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## 0.0.2 - WIP
+- Adding Rector PHP https://github.com/rectorphp/rector
+- Updating Readme for real-time updates
 
 ## 0.0.1 - 2024-04-06
 

composer.json

@@ -28,7 +28,8 @@
  "require-dev": {
  "laravel/pint": "^1.13",
  "orchestra/testbench": "^9.0",
- "pestphp/pest": "^2.34"
+ "pestphp/pest": "^2.34",
+ "rector/rector": "^1.0"
  },
  "autoload": {
  "psr-4": {

rector.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\TypeDeclaration\Rector\ClassMethod\AddVoidReturnTypeWhereNoReturnRector;
+
+return RectorConfig::configure()
+ ->withPaths([
+ __DIR__.'/config',
+ __DIR__.'/src',
+ ])
+ ->withPhpSets(
+ php83: true
+ )
+ ->withRules([
+ AddVoidReturnTypeWhereNoReturnRector::class,
+ ])
+ ->withPreparedSets(
+ deadCode: true,
+ codeQuality: true,
+ earlyReturn: true,
+ typeDeclarations: true,
+ codingStyle: true
+ );

src/FusionServiceProvider.php

@@ -9,7 +9,7 @@ class FusionServiceProvider extends ServiceProvider
  /**
  * Bootstrap the application services.
  */
- public function boot()
+ public function boot(): void
  {
  /*
  * Optional methods to load your package assets
@@ -47,14 +47,12 @@ public function boot()
  /**
  * Register the application services.
  */
- public function register()
+ public function register(): void
  {
  // Automatically apply the package configuration
  $this->mergeConfigFrom(__DIR__.'/../config/config.php', 'fusion');
 
  // Register the main class to use with the facade
- $this->app->singleton('fusion', function () {
- return new Fusion;
- });
+ $this->app->singleton('fusion', static fn (): \HiFolks\Fusion\Fusion => new Fusion);
  }
 }

src/Models/FusionBaseModel.php

@@ -20,7 +20,7 @@ public function getResourceFolder(): string
  $folderName = str_replace(
  'App\\Models\\',
  '',
- get_called_class()
+ static::class
  );
  $folderName = Str::snake($folderName);
 
@@ -39,14 +39,17 @@ public function frontmatterFields(): array
  ];
  }
 
- public function getRows()
+ public function getRows(): array
  {
  return $this->getFrontmatterRows(
  $this->frontmatterFields()
  );
  }
 
- public function getFrontmatterRows($columns = [])
+ /**
+ * @return non-empty-array[]
+ */
+ public function getFrontmatterRows($columns = []): array
  {
 
  $environment = (new Environment())

src/Traits/FusionModelTrait.php

@@ -8,7 +8,7 @@ trait FusionModelTrait
 {
  use Sushi;
 
- public function getRows()
+ public function getRows(): array
  {
  return parent::getRows();
  }

tests/Feature/ExampleTest.php

@@ -1,5 +1,5 @@
 <?php
 
-test('example', function () {
+test('example', function (): void {
  expect(true)->toBeTrue();
 });

tests/Pest.php

@@ -24,9 +24,7 @@
 |
 */
 
-expect()->extend('toBeOne', function () {
- return $this->toBe(1);
-});
+expect()->extend('toBeOne', fn () => $this->toBe(1));
 
 /*
 |--------------------------------------------------------------------------
@@ -39,7 +37,7 @@
 |
 */
 
-function something()
+function something(): void
 {
  // ..
 }

tests/Unit/ExampleTest.php

@@ -1,5 +1,5 @@
 <?php
 
-test('example', function () {
+test('example', function (): void {
  expect(true)->toBeTrue();
 });