avoid breaking change by removing void typehints

Author: freekmurze

.phpunit.cache/test-results

@@ -1 +1 @@
-{"version":"pest_2.36.0","defects":[],"times":{"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\AskToRunMigrationsTest::__pest_evaluable_it_can_ask_to_run_the_migrations":0.042,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\AssetsTest::__pest_evaluable_it_can_install_the_assets":0.009,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\ConfigFileTest::__pest_evaluable_it_can_install_the_config_file":0.001,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\CopyAndRegisterServiceProviderInAppTest::__pest_evaluable_it_can_copy_and_register_the_service_provider_in_the_app":0.003,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\EndWithTest::__pest_evaluable_it_can_execute_the_end_with":0.001,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\MigrationTest::__pest_evaluable_it_can_install_the_migrations":0.002,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\StarRepoTest::__pest_evaluable_it_can_propose_to_star_the_repo":0.001,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\StartWithTest::__pest_evaluable_it_can_execute_the_start_with":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageAssetsTest::__pest_evaluable_it_can_publish_the_assets":0.002,"P\\Tests\\PackageServiceProviderTests\\PackageCommandTest::__pest_evaluable_it_can_execute_a_registered_commands":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageCommandWithinAppTest::__pest_evaluable_it_can_execute_a_registered_command_in_the_context_of_the_app":0.011,"P\\Tests\\PackageServiceProviderTests\\PackageCommandsTest::__pest_evaluable_it_can_execute_a_registered_commands":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageConfigTest::__pest_evaluable_it_can_register_the_config_file":0,"P\\Tests\\PackageServiceProviderTests\\PackageConfigTest::__pest_evaluable_it_can_publish_the_config_file":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageDiscoversMigrationsTest::__pest_evaluable_it_publishes_discovered_migrations":0.003,"P\\Tests\\PackageServiceProviderTests\\PackageDiscoversMigrationsTest::__pest_evaluable_it_can_publish_the_migration_without_being_stubbed":0.002,"P\\Tests\\PackageServiceProviderTests\\PackageDiscoversMigrationsTest::__pest_evaluable_it_does_not_overwrite_the_existing_migration":0.004,"P\\Tests\\PackageServiceProviderTests\\PackageDiscoversMigrationsTest::__pest_evaluable_it_can_run_migrations_which_registers_them":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageMigrationTest::__pest_evaluable_it_can_publish_the_migration":0.002,"P\\Tests\\PackageServiceProviderTests\\PackageMigrationTest::__pest_evaluable_it_can_publish_the_migration_without_being_stubbed":0.002,"P\\Tests\\PackageServiceProviderTests\\PackageMigrationTest::__pest_evaluable_it_does_not_overwrite_the_existing_migration":0.003,"P\\Tests\\PackageServiceProviderTests\\PackageMigrationTest::__pest_evaluable_it_does_overwrite_the_existing_migration_with_force":0.003,"P\\Tests\\PackageServiceProviderTests\\PackageMigrationTest::__pest_evaluable_it_can_run_migrations_which_registers_them":0,"P\\Tests\\PackageServiceProviderTests\\PackageMultipleConfigTest::__pest_evaluable_it_can_register_multiple_config_files":0,"P\\Tests\\PackageServiceProviderTests\\PackageNameTest::__pest_evaluable_it_will_not_blow_up_when_a_name_is_set":0,"P\\Tests\\PackageServiceProviderTests\\PackagePublishableProviderTest::__pest_evaluable_it_can_publish_a_service_provider":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageRouteTest::__pest_evaluable_it_can_load_the_route":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageRouteTest::__pest_evaluable_it_can_load_multiple_route":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageRoutesTest::__pest_evaluable_it_can_load_the_route":0,"P\\Tests\\PackageServiceProviderTests\\PackageRoutesTest::__pest_evaluable_it_can_load_multiple_route":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageSharedDataTest::__pest_evaluable_it_can_share_data_with_all_views":0.005,"P\\Tests\\PackageServiceProviderTests\\PackageTranslationsTest::__pest_evaluable_it_can_load_the_translations":0.002,"P\\Tests\\PackageServiceProviderTests\\PackageTranslationsTest::__pest_evaluable_it_can_publish_the_translations":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageViewComponentsTest::__pest_evaluable_it_can_load_the_view_components":0.002,"P\\Tests\\PackageServiceProviderTests\\PackageViewComponentsTest::__pest_evaluable_it_can_publish_the_view_components":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageViewComposerTest::__pest_evaluable_it_can_load_the_view_composer_and_render_shared_data":0,"P\\Tests\\PackageServiceProviderTests\\PackageViewsTest::__pest_evaluable_it_can_load_the_views":0,"P\\Tests\\PackageServiceProviderTests\\PackageViewsTest::__pest_evaluable_it_can_publish_the_views":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageViewsWithCustomNamespaceTest::__pest_evaluable_it_can_load_the_views_with_a_custom_namespace":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageViewsWithCustomNamespaceTest::__pest_evaluable_it_can_publish_the_views_with_a_custom_namespace":0.001}}
+{"version":"pest_2.36.0","defects":[],"times":{"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\AskToRunMigrationsTest::__pest_evaluable_it_can_ask_to_run_the_migrations":0.043,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\AssetsTest::__pest_evaluable_it_can_install_the_assets":0.01,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\ConfigFileTest::__pest_evaluable_it_can_install_the_config_file":0.002,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\CopyAndRegisterServiceProviderInAppTest::__pest_evaluable_it_can_copy_and_register_the_service_provider_in_the_app":0.003,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\EndWithTest::__pest_evaluable_it_can_execute_the_end_with":0.001,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\MigrationTest::__pest_evaluable_it_can_install_the_migrations":0.002,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\StarRepoTest::__pest_evaluable_it_can_propose_to_star_the_repo":0.001,"P\\Tests\\PackageServiceProviderTests\\InstallCommandTests\\StartWithTest::__pest_evaluable_it_can_execute_the_start_with":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageAssetsTest::__pest_evaluable_it_can_publish_the_assets":0.002,"P\\Tests\\PackageServiceProviderTests\\PackageCommandTest::__pest_evaluable_it_can_execute_a_registered_commands":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageCommandWithinAppTest::__pest_evaluable_it_can_execute_a_registered_command_in_the_context_of_the_app":0.012,"P\\Tests\\PackageServiceProviderTests\\PackageCommandsTest::__pest_evaluable_it_can_execute_a_registered_commands":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageConfigTest::__pest_evaluable_it_can_register_the_config_file":0,"P\\Tests\\PackageServiceProviderTests\\PackageConfigTest::__pest_evaluable_it_can_publish_the_config_file":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageDiscoversMigrationsTest::__pest_evaluable_it_publishes_discovered_migrations":0.003,"P\\Tests\\PackageServiceProviderTests\\PackageDiscoversMigrationsTest::__pest_evaluable_it_can_publish_the_migration_without_being_stubbed":0.002,"P\\Tests\\PackageServiceProviderTests\\PackageDiscoversMigrationsTest::__pest_evaluable_it_does_not_overwrite_the_existing_migration":0.004,"P\\Tests\\PackageServiceProviderTests\\PackageDiscoversMigrationsTest::__pest_evaluable_it_can_run_migrations_which_registers_them":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageMigrationTest::__pest_evaluable_it_can_publish_the_migration":0.002,"P\\Tests\\PackageServiceProviderTests\\PackageMigrationTest::__pest_evaluable_it_can_publish_the_migration_without_being_stubbed":0.002,"P\\Tests\\PackageServiceProviderTests\\PackageMigrationTest::__pest_evaluable_it_does_not_overwrite_the_existing_migration":0.003,"P\\Tests\\PackageServiceProviderTests\\PackageMigrationTest::__pest_evaluable_it_does_overwrite_the_existing_migration_with_force":0.003,"P\\Tests\\PackageServiceProviderTests\\PackageMigrationTest::__pest_evaluable_it_can_run_migrations_which_registers_them":0,"P\\Tests\\PackageServiceProviderTests\\PackageMultipleConfigTest::__pest_evaluable_it_can_register_multiple_config_files":0,"P\\Tests\\PackageServiceProviderTests\\PackageNameTest::__pest_evaluable_it_will_not_blow_up_when_a_name_is_set":0,"P\\Tests\\PackageServiceProviderTests\\PackagePublishableProviderTest::__pest_evaluable_it_can_publish_a_service_provider":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageRouteTest::__pest_evaluable_it_can_load_the_route":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageRouteTest::__pest_evaluable_it_can_load_multiple_route":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageRoutesTest::__pest_evaluable_it_can_load_the_route":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageRoutesTest::__pest_evaluable_it_can_load_multiple_route":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageSharedDataTest::__pest_evaluable_it_can_share_data_with_all_views":0.004,"P\\Tests\\PackageServiceProviderTests\\PackageTranslationsTest::__pest_evaluable_it_can_load_the_translations":0.002,"P\\Tests\\PackageServiceProviderTests\\PackageTranslationsTest::__pest_evaluable_it_can_publish_the_translations":0.002,"P\\Tests\\PackageServiceProviderTests\\PackageViewComponentsTest::__pest_evaluable_it_can_load_the_view_components":0.002,"P\\Tests\\PackageServiceProviderTests\\PackageViewComponentsTest::__pest_evaluable_it_can_publish_the_view_components":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageViewComposerTest::__pest_evaluable_it_can_load_the_view_composer_and_render_shared_data":0,"P\\Tests\\PackageServiceProviderTests\\PackageViewsTest::__pest_evaluable_it_can_load_the_views":0,"P\\Tests\\PackageServiceProviderTests\\PackageViewsTest::__pest_evaluable_it_can_publish_the_views":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageViewsWithCustomNamespaceTest::__pest_evaluable_it_can_load_the_views_with_a_custom_namespace":0.001,"P\\Tests\\PackageServiceProviderTests\\PackageViewsWithCustomNamespaceTest::__pest_evaluable_it_can_publish_the_views_with_a_custom_namespace":0.001}}

src/PackageServiceProvider.php

@@ -45,7 +45,7 @@ public function newPackage(): Package
         return new Package();
     }
 
-    public function registerConfigs(): void
+    public function registerConfigs()
     {
         if (empty($this->package->configFileNames)) {
             return;
@@ -117,7 +117,7 @@ protected function bootAssets(): void
         $this->publishes([$vendorAssets => $appAssets], "{$this->package->shortName()}-assets");
     }
 
-    protected function bootCommands(): void
+    protected function bootCommands()
     {
         if (empty($this->package->commands)) {
             return;
@@ -126,7 +126,7 @@ protected function bootCommands(): void
         $this->commands($this->package->commands);
     }
 
-    protected function bootConsoleCommands(): void
+    protected function bootConsoleCommands()
     {
         if (empty($this->package->consoleCommands) || ! $this->app->runningInConsole()) {
             return;
@@ -135,7 +135,7 @@ protected function bootConsoleCommands(): void
         $this->commands($this->package->consoleCommands);
     }
 
-    protected function bootConfigs(): void
+    protected function bootConfigs()
     {
         if ($this->app->runningInConsole()) {
             foreach ($this->package->configFileNames as $configFileName) {
@@ -147,7 +147,7 @@ protected function bootConfigs(): void
         }
     }
 
-    protected function bootInertia(): void
+    protected function bootInertia()
     {
         if (! $this->package->hasInertiaComponents) {
             return;
@@ -166,7 +166,7 @@ protected function bootInertia(): void
         }
     }
 
-    protected function bootMigrations(): void
+    protected function bootMigrations()
     {
         if ($this->package->discoversMigrations) {
             $this->discoverMigrations();
@@ -198,7 +198,7 @@ protected function bootMigrations(): void
         }
     }
 
-    protected function bootProviders(): void
+    protected function bootProviders()
     {
         if (! $this->package->publishableProviderName || ! $this->app->runningInConsole()) {
             return;
@@ -211,7 +211,7 @@ protected function bootProviders(): void
         $this->publishes([$vendorProvider => $appProvider], "{$this->package->shortName()}-provider");
     }
 
-    protected function bootRoutes(): void
+    protected function bootRoutes()
     {
         if (empty($this->package->routeFileNames)) {
             return;
@@ -222,7 +222,7 @@ protected function bootRoutes(): void
         }
     }
 
-    protected function bootTranslations(): void
+    protected function bootTranslations()
     {
         if (! $this->package->hasTranslations) {
             return;
@@ -246,7 +246,7 @@ protected function bootTranslations(): void
         }
     }
 
-    protected function bootViews(): void
+    protected function bootViews()
     {
         if (! $this->package->hasViews) {
             return;
@@ -263,7 +263,7 @@ protected function bootViews(): void
         }
     }
 
-    protected function bootViewComponents(): void
+    protected function bootViewComponents()
     {
         if (empty($this->package->viewComponents)) {
             return;
@@ -281,7 +281,7 @@ protected function bootViewComponents(): void
         }
     }
 
-    protected function bootViewComposers(): void
+    protected function bootViewComposers()
     {
         if (empty($this->package->viewComposers)) {
             return;
@@ -292,7 +292,7 @@ protected function bootViewComposers(): void
         }
     }
 
-    protected function bootViewSharedData(): void
+    protected function bootViewSharedData()
     {
         if (empty($this->package->sharedViewData)) {
             return;
@@ -303,7 +303,7 @@ protected function bootViewSharedData(): void
         }
     }
 
-    protected function discoverMigrations(): void
+    protected function discoverMigrations()
     {
         $now = Carbon::now();
         $migrationsPath = trim($this->package->migrationsPath, '/');