Skip to content

Conversation

@Tobbe
Copy link
Member

@Tobbe Tobbe commented Jan 4, 2026

yarn clean is now what yarn build:clean used to be before #909 was merged

@netlify
Copy link

netlify bot commented Jan 4, 2026

Deploy Preview for cedarjs canceled.

Name Link
🔨 Latest commit b492140
🔍 Latest deploy log https://app.netlify.com/projects/cedarjs/deploys/695a33c6b892be0008a04cf7

@github-actions github-actions bot added this to the chore milestone Jan 4, 2026
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 4, 2026

Greptile Summary

Replaced yarn build:clean with yarn clean in test project fixture rebuild scripts. The change switches from a full nuclear clean (git clean + reinstall dependencies) to a lighter clean that only removes build artifacts (dist folders and tsconfig files). This optimization speeds up the rebuild process while still ensuring a clean build state.

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The change is a simple refactoring that replaces yarn build:clean with yarn clean in two test fixture scripts. Both commands are verified to exist in the root package.json. The change optimizes the rebuild process by switching from a full git clean to a lighter clean of build artifacts. No logic changes, no new dependencies, and both files have identical changes applied consistently.
  • No files require special attention

Important Files Changed

Filename Overview
tasks/test-project/rebuild-test-project-fixture-esm.ts Changed build command from yarn build:clean to yarn clean for faster rebuild process
tasks/test-project/rebuild-test-project-fixture.ts Changed build command from yarn build:clean to yarn clean for faster rebuild process

Sequence Diagram

sequenceDiagram
    participant Script as Rebuild Script
    participant Yarn as Yarn Package Manager
    participant Clean as tasks/clean.mjs
    participant Build as Build System
    
    Script->>Yarn: Execute yarn clean
    Yarn->>Clean: Run clean.mjs
    Clean->>Clean: Remove dist folders
    Clean->>Clean: Remove tsconfig.tsbuildinfo files
    Clean->>Clean: Run clean:prisma
    Clean-->>Yarn: Clean complete
    Yarn-->>Script: Clean complete
    
    Script->>Yarn: Execute yarn build
    Yarn->>Build: Trigger build process
    Build->>Build: Compile TypeScript
    Build->>Build: Generate build artifacts
    Build-->>Yarn: Build complete
    Yarn-->>Script: Build complete
Loading

@nx-cloud
Copy link

nx-cloud bot commented Jan 4, 2026

🤖 Nx Cloud AI Fix

Ensure the fix-ci command is configured to always run in your CI pipeline to get automatic fixes in future runs. For more information, please see https://nx.dev/ci/features/self-healing-ci


View your CI Pipeline Execution ↗ for commit b492140

Command Status Duration Result
nx run-many -t build:pack --exclude create-ceda... ✅ Succeeded 6s View ↗
nx run-many -t build ✅ Succeeded 6s View ↗
nx run-many -t test --minWorkers=1 --maxWorkers=4 ✅ Succeeded 3s View ↗
nx run-many -t test:types ✅ Succeeded 9s View ↗

☁️ Nx Cloud last updated this comment at 2026-01-04 09:49:17 UTC

@Tobbe Tobbe merged commit 3113e82 into main Jan 4, 2026
42 checks passed
@Tobbe Tobbe deleted the tobbe-chore-test-project-yarn-clean branch January 4, 2026 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants