[#92] Replaced 'deployment' terminology with 'artifact build' for clarity.

Author: AlexSkrypnyk

README.md

@@ -165,20 +165,24 @@ destination repository.
 
 ## 📥 Installation
 
-This package is intended to be used as a standalone binary. You will need to
+### As a standalone binary
+
+This tool is intended to be used as a standalone binary. You will need to
 have PHP installed on your system to run the binary.
 
 Download the latest release from the [GitHub releases page](https://github.com/drevops/git-artifact/releases/latest).
 
-You may also install the package globally using Composer:
+### As a Composer dependency
+
+You may also install this tool globally using Composer:
 ```shell
 composer global require --dev drevops/git-artifact:~1.1
 ```
 
-### 📌 Version constraint
+#### 📌 Version constraint
 
 When using `git-artifact` in CI/CD scripts, we recommend using **Tilde Version Range Operator** to ensure stability.
-The tilde constraint allows patch updates (e.g., `1.0.0` → `1.1.1`) but blocks minor version updates (e.g., `1.1.0` → `1.2.0`). 
+The tilde constraint allows patch updates (e.g., `1.0.0` → `1.1.1`) but blocks minor version updates (e.g., `1.1.0` → `1.2.0`).
 
 This ensures that:
 - **Security fixes and bug patches** are automatically applied

src/Commands/ArtifactCommand.php

@@ -107,17 +107,17 @@ class ArtifactCommand extends Command {
   protected bool $showChanges = FALSE;
 
   /**
-   * Flag to fail deployment when branch is missing.
+   * Flag to fail artifact packaging when branch is missing.
    *
-   * By default (false), deployment is skipped with exit code 0.
-   * When true, deployment fails with exit code 1.
+   * By default (false), artifact packaging is skipped with exit code 0.
+   * When true, artifact packaging fails with exit code 1.
    */
   protected bool $failOnMissingBranch = FALSE;
 
   /**
-   * Flag indicating deployment was skipped due to missing branch.
+   * Flag indicating artifact packaging was skipped due to missing branch.
    */
-  protected bool $deploymentSkipped = FALSE;
+  protected bool $packagingSkipped = FALSE;
 
   /**
    * Flag to specify if push was successful.
@@ -175,7 +175,7 @@ protected function configure(): void {
       ->addOption('root',                   NULL, InputOption::VALUE_REQUIRED, 'Path to the root for file path resolution. If not specified, current directory is used.')
       ->addOption('show-changes',           NULL, InputOption::VALUE_NONE,     'Show changes made to the repo by the build in the output.')
       ->addOption('src',                    NULL, InputOption::VALUE_REQUIRED, 'Directory where source repository is located. If not specified, root directory is used.')
-      ->addOption('fail-on-missing-branch', NULL, InputOption::VALUE_NONE,     'Fail deployment if source branch cannot be determined. By default, deployment is skipped gracefully.');
+      ->addOption('fail-on-missing-branch', NULL, InputOption::VALUE_NONE,     'Fail artifact packaging if source branch cannot be determined. By default, artifact packaging is skipped gracefully.');
     // @formatter:on
     // phpcs:enable Generic.Functions.FunctionCallArgumentSpacing.TooMuchSpaceAfterComma
     // phpcs:enable Drupal.WhiteSpace.Comma.TooManySpaces
@@ -209,8 +209,8 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
       $this->resolveOptions($remote, $input->getOptions());
 
-      // Check if deployment was skipped due to missing branch.
-      if ($this->deploymentSkipped) {
+      // Check if artifact packaging was skipped due to missing branch.
+      if ($this->packagingSkipped) {
         return Command::SUCCESS;
       }
 
@@ -225,7 +225,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
       return Command::FAILURE;
     }
 
-    $this->output->writeln('<info>Deployment finished successfully.</info>');
+    $this->output->writeln('<info>Artifact packaged successfully.</info>');
 
     return Command::SUCCESS;
   }
@@ -353,20 +353,20 @@ protected function resolveOptions(string $url, array $options): void {
     }
     catch (BranchNotFoundException $exception) {
       if ($this->failOnMissingBranch) {
-        // Strict mode: fail deployment.
-        throw new \RuntimeException('Unable to determine source branch. Deployment failed. ' . $exception->getMessage(), $exception->getCode(), $exception);
+        // Strict mode: fail artifact packaging.
+        throw new \RuntimeException('Unable to determine source branch. Artifact packaging failed. ' . $exception->getMessage(), $exception->getCode(), $exception);
       }
 
       $commit_hash = $exception->getCommitHash();
-      $this->logger->notice('Source branch not found. Deployment will be skipped.');
+      $this->logger->notice('Source branch not found. Artifact packaging will be skipped.');
       $this->logger->notice('Commit: ' . $commit_hash);
 
-      $this->output->writeln('<comment>Source branch not found. Deployment skipped.</comment>');
+      $this->output->writeln('<comment>Source branch not found. Artifact packaging skipped.</comment>');
       $this->output->writeln('<comment>Commit: ' . $commit_hash . '</comment>');
-      $this->output->writeln('<info>Use --fail-on-missing-branch to fail deployment instead.</info>');
+      $this->output->writeln('<info>Use --fail-on-missing-branch to fail artifact packaging instead.</info>');
 
-      // Set flag to skip deployment and return early from execute().
-      $this->deploymentSkipped = TRUE;
+      // Set flag to skip artifact packaging and return early from execute().
+      $this->packagingSkipped = TRUE;
 
       return;
     }

tests/Functional/FunctionalTestCase.php

@@ -113,7 +113,7 @@ protected function assertArtifactCommandSuccess(?array $args = [], string $branc
 
     $this->assertStringNotContainsString('[error]', $output);
     $this->assertStringContainsString(sprintf('Pushed branch "%s" with commit message "%s"', $branch, $commit), $output);
-    $this->assertStringContainsString('Deployment finished successfully.', $output);
+    $this->assertStringContainsString('Artifact packaged successfully.', $output);
     $this->assertStringNotContainsString('Processing failed with an error:', $output);
 
     return $output;
@@ -138,7 +138,7 @@ protected function assertArtifactCommandFailure(?array $args = [], string $commi
     $output = $this->runArtifactCommand($args, TRUE);
 
     $this->assertStringNotContainsString(sprintf('Pushed branch "%s" with commit message "%s"', $args['--branch'], $commit), $output);
-    $this->assertStringNotContainsString('Deployment finished successfully.', $output);
+    $this->assertStringNotContainsString('Artifact packaged successfully.', $output);
     $this->assertStringContainsString('Processing failed with an error:', $output);
 
     return $output;

tests/Functional/MissingBranchTest.php

@@ -14,10 +14,10 @@
 class MissingBranchTest extends FunctionalTestCase {
 
   /**
-   * Test deployment skips gracefully when branch is missing (default).
+   * Test artifact packaging skips gracefully when branch is missing (default).
    *
    * This simulates the scenario where a branch was deleted after PR merge.
-   * Default behavior should skip deployment with exit code 0.
+   * Default behavior should skip artifact packaging with exit code 0.
    */
   public function testMissingBranchDefaultBehavior(): void {
     $this->gitCreateFixtureCommits(1);
@@ -44,23 +44,23 @@ public function testMissingBranchDefaultBehavior(): void {
     // Checkout the orphaned commit - now it's truly not on any branch.
     $repo->checkout($commit_hash);
 
-    // Default behavior: should skip deployment with exit code 0.
+    // Default behavior: should skip artifact packaging with exit code 0.
     // Use explicit branch name (no tokens) to avoid token processing issues.
     $output = $this->runArtifactCommand([
       '--branch' => 'testbranch',
       '--dry-run' => TRUE,
     ]);
 
-    $this->assertStringContainsString('Source branch not found. Deployment skipped.', $output);
+    $this->assertStringContainsString('Source branch not found. Artifact packaging skipped.', $output);
     $this->assertStringContainsString('Commit: ' . $commit_hash, $output);
-    $this->assertStringContainsString('Use --fail-on-missing-branch to fail deployment instead', $output);
+    $this->assertStringContainsString('Use --fail-on-missing-branch to fail artifact packaging instead', $output);
     $this->assertStringNotContainsString('Processing failed with an error:', $output);
   }
 
   /**
-   * Test deployment fails when branch is missing with flag.
+   * Test artifact packaging fails when branch is missing with flag.
    *
-   * With --fail-on-missing-branch flag, deployment should fail.
+   * With --fail-on-missing-branch flag, artifact packaging should fail.
    */
   public function testMissingBranchWithFlag(): void {
     $this->gitCreateFixtureCommits(1);
@@ -100,29 +100,29 @@ public function testMissingBranchWithFlag(): void {
   }
 
   /**
-   * Test normal deployment when branch exists.
+   * Test normal artifact packaging when branch exists.
    *
    * Verifies that normal operation still works when branch is available.
    */
   public function testNormalDeploymentWithBranch(): void {
     $this->gitCreateFixtureCommits(1);
 
-    // Normal deployment with existing branch should work.
+    // Normal artifact packaging with existing branch should work.
     $output = $this->assertArtifactCommandSuccess();
 
     $this->assertStringContainsString('Pushed branch "testbranch" with commit message "Deployment commit"', $output);
-    $this->assertStringContainsString('Deployment finished successfully.', $output);
+    $this->assertStringContainsString('Artifact packaged successfully.', $output);
 
     // Verify the branch exists in destination.
     $this->gitCheckout($this->dst, 'testbranch');
     $this->assertFilesExist($this->dst, 'f1');
   }
 
   /**
-   * Test deployment works with tag in detached HEAD state.
+   * Test artifact packaging works with tag in detached HEAD state.
    *
    * When checked out at a tag, getOriginalBranch() should validate that
-   * the tag exists and allow deployment to proceed.
+   * the tag exists and allow artifact packaging to proceed.
    */
   public function testDeploymentWithTagDetachedHead(): void {
     $this->gitCreateFixtureCommits(1);
@@ -135,11 +135,11 @@ public function testDeploymentWithTagDetachedHead(): void {
     // Checkout the tag to enter detached HEAD state.
     $repo->checkout('v1.0.0');
 
-    // Deployment should work because tag is a valid detachment source.
+    // Artifact packaging should work because tag is a valid detachment source.
     $output = $this->assertArtifactCommandSuccess();
 
     $this->assertStringContainsString('Pushed branch "testbranch" with commit message "Deployment commit"', $output);
-    $this->assertStringContainsString('Deployment finished successfully.', $output);
+    $this->assertStringContainsString('Artifact packaged successfully.', $output);
   }
 
 }