nushell / nufmt

MIT License
64 stars 8 forks source link

Split lib and bin into crates and minor refactoring #49

Open IogaMaster opened 10 months ago

IogaMaster commented 10 months ago

this will close #48

changelog

Justification

  1. Improved Readability: Decoupling the code into modular packages can enhance code readability. It allows developers to focus on one specific aspect of the code at a time, making it easier to understand.

  2. Reduced Cognitive Load: With decoupled packages, developers don't need to grasp the entire codebase in one go. They can delve into specific modules as needed, reducing the cognitive load and making it more accessible.

  3. Easier Onboarding: When new team members join nufmt, a decoupled codebase makes it easier for them to get up to speed.

  4. Maintainability: A more understandable codebase is inherently more maintainable. Developers can make changes and improvements more confidently, reducing the risk of introducing new issues.

  5. Long-term Viability: A codebase that is hard to understand can become a liability in the long run. By investing in a rewrite now, we can ensure the long-term viability and sustainability of nufmt.

IogaMaster commented 10 months ago

Alright all basic functionality is restored in this pr. Please test and let me know what else is in need of changing.

I still need to work on: