Skip to content

fix(locadex): Fix matching files default#384

Merged
brian-lou merged 2 commits intomainfrom
b/locadex/fix-matching-files-default
Jun 14, 2025
Merged

fix(locadex): Fix matching files default#384
brian-lou merged 2 commits intomainfrom
b/locadex/fix-matching-files-default

Conversation

@brian-lou
Copy link
Copy Markdown
Contributor

No description provided.

@brian-lou brian-lou requested a review from a team as a code owner June 14, 2025 03:11
@vercel
Copy link
Copy Markdown

vercel bot commented Jun 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gt-docs 🔄 Building (Inspect) Visit Preview 💬 Add feedback Jun 14, 2025 3:11am
5 Skipped Deployments
Name Status Preview Comments Updated (UTC)
example-ai-chatbot ⬜️ Ignored (Inspect) Jun 14, 2025 3:11am
example-create-react-app ⬜️ Ignored (Inspect) Jun 14, 2025 3:11am
example-gt-next-starter ⬜️ Ignored (Inspect) Jun 14, 2025 3:11am
example-next-create-app ⬜️ Ignored (Inspect) Jun 14, 2025 3:11am
example-vite-create-app ⬜️ Ignored (Inspect) Jun 14, 2025 3:11am

@brian-lou brian-lou merged commit 1690a7b into main Jun 14, 2025
7 of 8 checks passed
@brian-lou brian-lou deleted the b/locadex/fix-matching-files-default branch June 14, 2025 03:11
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

Fixed a critical path resolution issue in Locadex configuration where file matching patterns would become invalid when rootDir and appDir were identical.

  • Modified packages/locadex/src/utils/config.ts to handle empty relative paths by using '.' as fallback, ensuring glob patterns like './**/*.{ts,tsx,js,jsx}' work correctly
  • Prevents invalid glob pattern generation when path.relative() returns empty string

1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile

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.

1 participant