Skip to content

Commit b59cdab

Browse files
Refactor Bootstrap Units into Separate Class for Improved Organization
This update introduces a new `BootstrapUnits` class to encapsulate all bootstrap units used during initialization, thereby reducing clutter in the `PhysicalDimensions` class. The refactor enhances code organization and maintainability by centralizing the definition of bootstrap units, which are essential for breaking circular dependencies. The `PhysicalDimensions` class has been updated to reference these units, streamlining the overall structure of the codebase.
1 parent 6abf3df commit b59cdab

4 files changed

Lines changed: 1054 additions & 131 deletions

File tree

.cursor/rules/derived-cursor-rules.mdc

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,4 +221,5 @@ When adding XML documentation comments:
221221
* Mathematical constants (like `Math.PI`): Already using C# standard library
222222
* Derived unit relationships (like time conversions): `60.0` (seconds per minute), `3600.0` (seconds per hour)
223223
* Imperial unit conversions not in PhysicalConstants: Many specific imperial conversions that aren't fundamental physical constants
224-
* **Testing Derived Constants:** Add additional tests for physical constants to validate stored values against calculated values from fundamental constants.
224+
* **Testing Derived Constants:** Add additional tests for physical constants to validate stored values against calculated values from fundamental constants.
225+
* Can we put the bootstrap units in their own class so they dont pollote the PhysicalDimensions namespace?

0 commit comments

Comments
 (0)