Remove stray full stops from end of lines in inline comments (#185)

I've made a script to autofix all PHPCS complaints related to "all
comments must end with a full stop" since the sniff complains but does
not autofix. It seems like I haven't considered multiline comments :-)
This PR removes a bunch of end-of-line full stops from places where they
don't make any sense.

Author: adamziel

components/Blueprints/DataReference/class-datareferenceresolver.php

@@ -81,9 +81,9 @@ public function resolve( DataReference $reference ) {
 		return $this->resolved_data_references[ $reference->id ];
 	}
 
-	// @TODO: Clean up the semantics of this class. Resolve() and separate resolve_uncached() seem confusing. There's.
-	// a bunch of implicit behaviors related to caching. Ideally we would either have a self-contained resolution.
-	// method, or co-locate the resolution logic with the data reference classes and only use this class for.
+	// @TODO: Clean up the semantics of this class. Resolve() and separate resolve_uncached() seem confusing. There's
+	// a bunch of implicit behaviors related to caching. Ideally we would either have a self-contained resolution
+	// method, or co-locate the resolution logic with the data reference classes and only use this class for
 	// caching.
 	public function resolve_uncached( DataReference $reference ) {
 		$progress_tracker = $this->sub_trackers[ $reference->id ] ?? new Tracker();
@@ -123,7 +123,7 @@ public function resolve_uncached( DataReference $reference ) {
 				$tracked_stream,
 				$filename
 			);
-			// TODO: Consider a clearer name. Some not-so-great ballpark ideas:.
+			// TODO: Consider a clearer name. Some not-so-great ballpark ideas:
 			// BlueprintParentPath, BlueprintRootPath, BlueprintContextPath, BlueprintRelativePath.
 		} elseif ( $reference instanceof ExecutionContextPath ) {
 			$path = $reference->get_path();

components/Blueprints/Progress/class-progresstrackedreadstream.php

@@ -76,7 +76,7 @@ public function consume_all(): string {
 
 	public function close_reading(): void {
 		$this->stream->close_reading();
-		// Optionally ensure tracker is set to 100% if not already.
+		// Optionally ensure tracker is set to 100% if not already
 		// This depends on whether closing implies completion.
 		// For now, we rely on consume_all or reaching end of data.
 		// If the stream is closed prematurely, the progress will reflect the last read amount.
@@ -102,7 +102,7 @@ private function updateProgress(): void {
 		}
 
 		$progress = ( $this->stream->tell() / $this->stream_length ) * 100;
-		// It's possible to seek() backwards. Let's make sure we never decrease.
+		// It's possible to seek() backwards. Let's make sure we never decrease
 		// the reported progress.
 		if ( $progress > $this->tracker->getProgress() ) {
 			$this->tracker->set( $progress );

components/Blueprints/SiteResolver/class-newsiteresolver.php

@@ -104,9 +104,9 @@ public static function resolve( Runtime $runtime, Tracker $progress, ?VersionCon
 		}
 
 		// 3. Install WordPress if not installed yet.
-		// Technically, this is a "new site" resolver, but it's entirely possible.
-		// the developer-provided WordPress zip already has a sqlite database with the.
-		// a WordPress site installed..
+		// Technically, this is a "new site" resolver, but it's entirely possible
+		// the developer-provided WordPress zip already has a sqlite database with
+		// a WordPress site installed.
 		if ( ! self::is_wordpress_installed( $runtime, $progress ) ) {
 			if ( ! $target_fs->exists( '/wp-config.php' ) ) {
 				if ( $target_fs->exists( 'wp-config-sample.php' ) ) {
@@ -128,7 +128,7 @@ public static function resolve( Runtime $runtime, Tracker $progress, ?VersionCon
 					'install',
 					'--path=' . $runtime->get_configuration()->get_target_site_root(),
 
-					// For Docker compatibility. If we got this far, Blueprint runner was already.
+					// For Docker compatibility. If we got this far, Blueprint runner was already
 					// allowed to run as root.
 					'--allow-root',
 					'--url=' . $runtime->get_configuration()->get_target_site_url(),
@@ -222,7 +222,7 @@ function ( $v ) {
 				preg_match( '/^\d+\.\d+$/', $version_string ) &&
 				$version_string === $api_version['partial_version']
 			) {
-				// When the Blueprint provides a version like 6.6, we must match on the partial.
+				// When the Blueprint provides a version like 6.6, we must match on the partial
 				// version, e.g. "6.6".
 				return $api_version['download'];
 			}

components/Blueprints/Steps/class-activatepluginstep.php

@@ -9,7 +9,7 @@
  * Represents the 'activatePlugin' step.
  */
 class ActivatePluginStep implements StepInterface {
-	// Inline PHP script to avoid reading a static script.php file via.
+	// Inline PHP script to avoid reading a static script.php file via
 	// file_get_contents() inside the built blueprints.phar file.
 	const ACTIVATE_PLUGIN_SCRIPT = <<<'PHP'
 <?php

components/Blueprints/Steps/class-activatethemestep.php

@@ -10,7 +10,7 @@
  */
 class ActivateThemeStep implements StepInterface {
 
-	// Inline PHP script to avoid reading a static script.php file via.
+	// Inline PHP script to avoid reading a static script.php file via
 	// file_get_contents() inside the built blueprints.phar file.
 	const ACTIVATE_THEME_SCRIPT = <<<'PHP'
 <?php

components/Blueprints/Steps/class-defineconstantsstep.php

@@ -28,7 +28,7 @@ public function __construct( array $constants ) {
 	 */
 	public function run( Runtime $runtime, Tracker $tracker ) {
 		$tracker->setCaption( 'Defining wp-config constants' );
-		// Inline PHP script to avoid reading a static script.php file via.
+		// Inline PHP script to avoid reading a static script.php file via
 		// file_get_contents() inside the built blueprints.phar file.
 		$runtime->eval_php_code_in_subprocess(
 			<<<'PHP'
@@ -296,7 +296,7 @@ function rewrite_wp_config_to_define_constants( $content, $constants = array() )
 			continue;
 		}
 
-		// We now have a define() call that defines a constant we're looking for.
+		// We now have a define() call that defines a constant we're looking for
 		// Let's rewrite its value to the one
 		$output = array_merge(
 			$output,

components/Blueprints/Steps/class-importcontentstep.php

@@ -89,7 +89,7 @@ private function importWxr( Runtime $runtime, array $content_definition, Tracker
 		foreach ( $this->output_lines( $output ) as $line ) {
 			$data = @json_decode( $line, true );
 			if ( ! is_array( $data ) ) {
-				// Non-JSON output is treated as a crash. We use a dedicated file pipe.
+				// Non-JSON output is treated as a crash. We use a dedicated file pipe
 				// for communication and it should never contain a non-JSON line.
 				$import_process->stop();
 				throw new ProcessFailedException( $import_process );

components/Blueprints/Steps/class-importmediastep.php

@@ -107,9 +107,9 @@ function ( $media ) {
 				if ( ! $resolved instanceof File ) {
 					// TODO: What if the schema specifies a resource type that can only be resolved to a file?
 					// Then the resolver could throw the error instead of requiring each step to check.
-					// But since the resolve interface can return either a File or Directory,.
+					// But since the resolve interface can return either a File or Directory,
 					// would we have to check anyway?
-					// Would there be any value in the runtime having specific methods like resolveFile().
+					// Would there be any value in the runtime having specific methods like resolveFile()
 					// and resolveDirectory() that throw if they cannot resolve the requested type?
 					throw new RuntimeException( "Failed to resolve media file: $human_readable_name" );
 				}

components/Blueprints/Steps/class-importthemestartercontentstep.php

@@ -23,7 +23,7 @@ public function __construct( ?string $theme_slug = null ) {
 
 	public function run( Runtime $runtime, Tracker $tracker ) {
 		$tracker->setCaption( 'Importing theme starter content' . ( $this->theme_slug ? ' for ' . $this->theme_slug : '' ) );
-		// Inline PHP script to avoid reading a static script.php file via.
+		// Inline PHP script to avoid reading a static script.php file via
 		// file_get_contents() inside the built blueprints.phar file.
 		$runtime->eval_php_code_in_subprocess(
 			<<<'PHP'

components/Blueprints/Steps/class-installthemestep.php

@@ -93,7 +93,7 @@ function ( $temp_dir ) use ( $runtime, $tracker ) {
 
 				$tracker->set( 50 );
 
-				// Inline PHP script to avoid reading a static script.php file via.
+				// Inline PHP script to avoid reading a static script.php file via
 				// file_get_contents() inside the built blueprints.phar file.
 				$output = $runtime->eval_php_code_in_subprocess(
 					<<<'PHP'