Restructure project layout#35
Conversation
|
Important Review skippedToo many files! This PR contains 151 files, which is 1 over the limit of 150. To get a review, narrow the scope: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (6)
📒 Files selected for processing (151)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Arctis-Fireblight
left a comment
There was a problem hiding this comment.
I think from a structure standpoint it looks like a vast improvement.
There is still much to do in regards to renaming, but I think this is a good start for us to start building off of.
Approved!
Restructure only the root directory structure for now.
Also includes:
Doctest header that pre-defines the entry point code, and with extra macros.
Fix Shaders.cmake (was using a hard-coded target name).
Clean up includes and imports across core.
Clean up Vector types code, including fixing the non-const consteval select cases.
Make Transform functions member functions, add overloads that use Vector3.