Skip to content

Conversation

@mondoreale
Copy link
Contributor

@mondoreale mondoreale commented Dec 8, 2025

This pull request updates TypeScript configuration and build scripts for the cdn-location (primarily) and dht packages to improve type checking, project references, and test configuration. The main focus is on refining how different parts of the codebase are built and checked, especially for data generation and testing.

Changes

TypeScript configuration improvements:

  • Added a new tsconfig.data-generation.json in cdn-location for type-checking the data-generation directory without emitting output, and updated the main tsconfig.json to reference it.
  • Updated tsconfig.jest.json in cdn-location to only include the test directory, and changed project references to point to the local tsconfig.node.json and test-utils.
  • Adjusted tsconfig.node.json in cdn-location to remove the reference to test-utils, keeping only utils as a reference.

Build and check script enhancements:

  • Improved the check script in cdn-location/package.json to run type checks for both test and data generation configs, ensuring stricter type safety across the project.

Project references update for dht package:

  • Changed dht package TypeScript configs to reference the entire cdn-location project instead of just its tsconfig.node.json, simplifying dependency management.

@linear
Copy link

linear bot commented Dec 8, 2025

@github-actions github-actions bot added the dht Related to DHT package label Dec 8, 2025
@mondoreale mondoreale changed the title refactor [NET-1644]: Clean-up tsconfigs for the cdn-location package refactor [NET-1645]: Clean-up tsconfigs for the cdn-location package Dec 8, 2025
@mondoreale mondoreale force-pushed the net-1645-clean-up-cdn-location-package-tsconfigs branch from f0a5cb0 to a900f1d Compare December 12, 2025 00:50
@mondoreale mondoreale merged commit 1c39b80 into main Dec 12, 2025
42 of 43 checks passed
@mondoreale mondoreale deleted the net-1645-clean-up-cdn-location-package-tsconfigs branch December 12, 2025 01:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dht Related to DHT package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants