Skip to content

fix: rename TSConfig.json to tsconfig.json - #82

Open
erossignon wants to merge 1 commit into
SerafinTech:masterfrom
erossignon:fix/tsconfig-lowercase-filename
Open

fix: rename TSConfig.json to tsconfig.json#82
erossignon wants to merge 1 commit into
SerafinTech:masterfrom
erossignon:fix/tsconfig-lowercase-filename

Conversation

@erossignon

Copy link
Copy Markdown

Summary

Renames TSConfig.json to the exact lowercase tsconfig.json that tsc and bundlers expect. Fixes #81.

Why

tsc (the build script runs plain tsc) and esbuild look for the exact filename tsconfig.json. On case-insensitive filesystems (Windows, macOS) the mis-cased file happens to be picked up, but on Linux:

  • tsc silently compiles with default options instead of the settings in this file, and
  • downstream consumers that bundle this package with esbuild fail hard with Cannot find tsconfig file "node_modules/st-ethernet-ip/tsconfig.json" (that is how we hit this — our CI packaging step works on Windows dev machines and dies on the Linux runner).

Renaming makes every platform behave identically. No content change — git records it as a 100% rename.

🤖 Generated with Claude Code

tsc and bundlers such as esbuild look for the exact lowercase filename
tsconfig.json. On case-insensitive filesystems (Windows, macOS) the
mis-cased file happens to be found, but on Linux the lookup fails: tsc
falls back to compiling with default options, and esbuild-based
consumers that bundle this package error out with 'Cannot find tsconfig
file'. Renaming the file makes every platform behave identically.

Fixes SerafinTech#81
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.

TSConfig.json should be tsconfig.json

1 participant