Skip to content

Conversation

@JohnC-80
Copy link
Contributor

@JohnC-80 JohnC-80 commented Nov 17, 2025

On windows 😝
path.join(txPath, '*.json')
results in

path\\that\\looks\\like\\this\\*.json

fast-glob reportedly needs

path/that/looks/like/this/*.json

EDIT: Just switched to using fs.globSync and canned the dep.

@JohnC-80 JohnC-80 requested a review from a team as a code owner November 17, 2025 14:29
@github-actions
Copy link

github-actions bot commented Nov 17, 2025

Jest Unit Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 688d230. ± Comparison against base commit 42873e4.

♻️ This comment has been updated with latest results.

Copy link
Member

@zburke zburke left a comment

Choose a reason for hiding this comment

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

LGTM but

  1. We should figure out if fast-glob is really even necessary here or if we could as easily grab stuff with straight up node APIs, or with the formatjs APIs.
  2. Please file a STCLI story and stick it in the PR title before you merge so the jira<->github magic linker can find it in the commit log and keep them sync'ed. Having the Jira paperwork in order is helpful when managing official releases.

@JohnC-80 JohnC-80 changed the title Translate compile on Windows - fast-glob needs posix separators. STCLI-273 Translate compile on Windows - cross-env globbing. Nov 19, 2025
Copy link
Member

@zburke zburke left a comment

Choose a reason for hiding this comment

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

Even better, thank you!

@sonarqubecloud
Copy link

@JohnC-80 JohnC-80 merged commit 69a9ca3 into main Nov 20, 2025
15 checks passed
@JohnC-80 JohnC-80 deleted the translate-compile-posix branch November 20, 2025 14:42
JohnC-80 added a commit that referenced this pull request Nov 20, 2025
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.

3 participants