chore: remove tracked build artifacts and fix CI/CD pipeline - #89
Merged
Conversation
- Remove .next/ from git tracking (926 build cache files) and add to .gitignore - Remove packages/validators/node_modules/ from tracking (symlinks) - Remove placeholder directories: your-repository-name-here/, store-assets/ - Remove stale artifacts: DEPLOY_TRIGGER.md, playwright-report/, test-results/ - Remove accidentally committed analytics-dashboard/ (separate project) - Add .next/, out/, playwright-report/, test-results/ to .gitignore - Fix web-ci.yml: remove npm cache reference (project uses pnpm) - Fix release.yml: replace npm ci with pnpm install --frozen-lockfile, add pnpm/action-setup - Fix eslint.config.js: remove react-native/no-inline-styles rules (plugin not installed) - Fix apps/web/package.json: downgrade ESLint from ^10.4.1 to ^9.39.4 (eslint-plugin-react compat) - Fix apps/web/src/app/demo/page.tsx: fix React Compiler set-state-in-effect error - Fix apps/web/src/lib/clerk/useAuth.ts: fix React Compiler set-state-in-effect error
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Removes 954 files of tracked build artifacts, stale directories, and placeholder repos. Fixes CI/CD pipeline and ESLint configuration to pass cleanly.
Changes
Tracked artifacts removed from git
.next/— 926 build cache files (Turbopack, SSR chunks, fonts, dev traces)packages/validators/node_modules/— 5 tracked symlinksanalytics-dashboard/— 9 files (separate project accidentally committed)test-results/— 9 files (GitHub templates, README, LICENSE)playwright-report/— 1 tracked fileyour-repository-name-here/— empty placeholder directorystore-assets/— template READMEDEPLOY_TRIGGER.md— deployment timestamp artifact.gitignore updated
Added:
.next/,out/,playwright-report/,test-results/CI/CD fixes
cache: 'npm'(project uses pnpm)npm ciwithpnpm install --frozen-lockfile, addedpnpm/action-setup@v6, removed npm cache referencesESLint fixes
react-native/no-inline-stylesrules (plugin not installed)^10.4.1to^9.39.4(eslint-plugin-react compatibility)React Compiler errors fixed
Result
ESLint: 0 errors, 42 warnings (all pre-existing). Exit code 0.
Risk
No runtime behavior changes. Demo page and auth hook maintain identical functionality with cleaner patterns. All remaining warnings are pre-existing (unused vars, any types, etc.).