fix: Removes redundant build step (#581)

* fix: Removes redundant build step

* Update release.yml

Makes git add command more specific

* fix: Update playwright.yml

Adds steps.get_workspaces.outputs.affected_workspaces != '' to the test step. This will prevent the job from failing when there are no changes to test.

Author: willum070

.github/workflows/playwright.yml

@@ -106,6 +106,7 @@ jobs:
         #   contains(github.event.pull_request.paths.*, 'samples/generate-index.sh') # Or if the script itself changed
 
       - name: Run All Playwright Tests
+        if: steps.get_workspaces.outputs.affected_workspaces != ''
         run: npx playwright test e2e/samples.spec.ts
         env:
           # Pass the correct base reference for find-changes.sh when called by samples.spec.ts

.github/workflows/release.yml

@@ -110,7 +110,7 @@ jobs:
 
       - name: Commit and Push Changes (to temp branch)
         run: |
-          git add -A
+          git add dist/ package-lock.json
           if [ -z "$(git status --porcelain)" ]; then
             echo "No changes to commit"
             exit 0

e2e/samples.spec.ts

@@ -107,26 +107,6 @@ if (foldersToTest.length === 0) {
 foldersToTest.forEach((sampleFolder) => {
   test(`test ${sampleFolder}`, async ({ page }) => {
 
-    // START Build the sample
-    const buildProcess = childProcess.spawn('npm', ['run', 'build'], {
-      cwd: path.join(samplesDir, sampleFolder),
-      stdio: 'inherit',
-    });
-
-    await new Promise((resolve, reject) => {
-        buildProcess.on('close', (code) => {
-          if (code === 0) {
-            resolve(true);
-          } else {
-            reject(new Error(`Build process for ${sampleFolder} exited with code ${code}`));
-          }
-        });
-        buildProcess.on('error', (err) => {
-            reject(new Error(`Failed to start build process for ${sampleFolder}: ${err.message}`));
-        });
-      });
-    // END Build the sample
-
     // START run the preview
     // Get an available port
     const port = 8080;