Improvement Suggestions for the Repository

MDReal32 commented 11 months ago

Hello! I've been following your content on YouTube and really appreciate the valuable information you share. Thank you for that!

I've taken the liberty to enhance the repository in a few ways:

Directory Structure Improvement: I've restructured the directories to make the project more organized and easier to navigate.
Build Script Enhancement: I've modified the build script and included a comment that updates the exports field in the package.json file. While I understand this might not align with best practices, I've kept this feature intact, leaving the decision to you. As an alternative, you might consider removing the exports field from package.json and allowing the CI process to generate it based on inputs.
Simplified Imports: Users can now import specific section using the following syntax: import ".../
"; This makes imports more straightforward.
```
import "@total-typescript/ts-reset/array";
// or
import "@total-typescript/ts-reset/set";
```

These improvements aim to enhance the readability, maintainability, and usability of the repository. Please feel free to review these changes and adapt them according to your preferences. Your input is always valued. Thanks again for your fantastic content!

NateLevin1 commented 11 months ago

Was the description of this PR generated by ChatGPT? These changes are pretty substantial (almost entirely breaking changes), you may just want to keep your own fork instead of merging with this repository.

MDReal32 commented 11 months ago

No. Its not generated by ChatGPT. just fixed my grammatical problems. Yeah it braking changes. It can be released in next major update.

mattpocock commented 2 months ago

Apologies, this PR is too large to merge. Individually, we could discuss these pieces and get them merged one by one. But I'm unsure about each of them:

I'm happy with the directory structure the way it is.

I don't want to change the way users import this library, this seems to be an unnecessary breaking change.

There is an argument that we should be automatically generating the exports map - but I also quite like keeping it simple.

total-typescript / ts-reset

Improvement Suggestions for the Repository #162