Skip to content

Conversation

@mondoreale
Copy link
Contributor

This pull request mainly makes formatting and minor configuration changes to various TypeScript configuration files across several packages. The changes primarily involve expanding single-line arrays for types and include fields into multi-line arrays for improved readability, and removing some unused or redundant compiler options.

Changes

These changes do not affect application logic but help keep configuration files clean and maintainable.

Configuration formatting improvements:

  • Reformatted the types and include arrays in multiple tsconfig.*.json files (such as tsconfig.karma.json and tsconfig.jest.json) across dht, proto-rpc, trackerless-network, test-utils, and utils packages to use multi-line arrays for better readability. [1] [2] [3] [4] [5]

Removal of unused or redundant compiler options:

  • Removed the noEmit compiler option from several tsconfig.jest.json files where it was unnecessary, such as in dht, proto-rpc, sdk, and trackerless-network packages. [1] [2] [3] [4]
  • Removed the resolveJsonModule compiler option from sdk/tsconfig.jest.json and sdk/tsconfig.node.json as it was defined upstream. [1] [2]

@github-actions github-actions bot added network Related to Network Package test-utils Related to Test Utils Package dht Related to DHT package utils proto-rpc sdk labels Dec 14, 2025
@mondoreale
Copy link
Contributor Author

Will get this in – makes for a few changes spread in individual PRs and is a noop logic-wise.

@mondoreale mondoreale merged commit 2281fa2 into main Dec 14, 2025
23 checks passed
@mondoreale mondoreale deleted the tsconfig-cosmetics branch December 14, 2025 16:50
mondoreale added a commit that referenced this pull request Dec 14, 2025
This pull request makes minor formatting adjustments to the
`tsconfig.karma.json` file, expanding the `types` and `include` fields
into multi-line arrays for improved readability. No functional changes
were made.[^1]

[^1]: It also is a follow-up to #3293 – I missed a spot.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dht Related to DHT package network Related to Network Package proto-rpc sdk test-utils Related to Test Utils Package utils

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants