Skip to content

✨ Adding C++20 Modules #168

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
SpinnerX merged 68 commits into from
Jan 23, 2026
Merged

✨ Adding C++20 Modules #168

SpinnerX merged 68 commits into from
Jan 23, 2026

Conversation

@SpinnerX
Copy link
Member

@SpinnerX SpinnerX commented Jan 19, 2026 •
edited
Loading

This PR resolves #167

With additional changes in updating the CI.

SpinnerX added 30 commits January 10, 2026 14:22
@SpinnerX
WIP migrating to cpp modules
9883690
@SpinnerX
WIP modules migration and re-adding logger
4e54a9a
@SpinnerX
WIP continuing modules relationships
50ec735
@SpinnerX
WIP in re-implementing the editor with modules
e71eff1
@SpinnerX
WIP in re-adding vulkan instance initialization
6fd98ec
@SpinnerX
WIP windowing and GLFW initialization!
98a7865
@SpinnerX
WIP ensuring the vulkan instances and windowing system are cleaned up…
5a50687 
… properly
@SpinnerX
WIP re-added event.cppm to target_sources in project cmake
b087b34
@SpinnerX
WIP in re-adding vulkan device handles and fixed includes for specify…
a428389 
…ing platform-specific vulkan extensions
@SpinnerX
WIP in re-adding registration callbacks + more work to do in getting …
9d4160e 
…vulkan implementation working again
@SpinnerX
WIP in re-adding in the game state API's and fixed the logical device…
3c82d5f 
… cleanup to be correct
@SpinnerX
WIP -- re-adding handling event inputs
6496c47
@SpinnerX
WIP -- re-adding more events implementations
ffef98f
@SpinnerX
WIP -- re-added swapchain implementation
8439e4e
@SpinnerX
WIP -- finishing up swapchain and window_context implementation
786a0b3
@SpinnerX
WIP -- re-added math functions, and the scene system
8c915d9
@SpinnerX
WIP -- re-added the vulkan implementation-specific renderer back with…
005be81 
… modules applied
@SpinnerX
WIP -- getting level scene to work like normal and getting the render…
a3dbd00 
…er to work again
@SpinnerX
WIP -- level scene working again
eaa1730
@SpinnerX
WIP -- re-
06cb85a 
added physics components
@SpinnerX
WIP -- re-added the UI API's and fixed aspect ratio for window resizi…
78b762e 
…ng, and added the editor logic for UI
@SpinnerX
WIP -- re-added serialization system for loading/saving scenes
4b89fb9
@SpinnerX
WIP -- re-added physics simulation to work again
1ed7121
@SpinnerX
WIP -- added another game object to level_scene
222baa8
@SpinnerX
WIP -- made the entry point automated to be internal
1e705d7
@SpinnerX
WIP -- cleaned up editor/application.cpp
a959a6c
@SpinnerX
Removed old atlas headers project structure
5fba872
@SpinnerX
Removed old editor code that uses headers
f2ebaf3
@SpinnerX
Renamed atlas-modules to atlas to finalize changes migrating to modules
617ebe5
@SpinnerX
Renamed from editor_modules to editor to finalize changes migrating e…
6491b14 
…ditor code to modules
@SpinnerX SpinnerX changed the title Cpp modules ✨ Adding C++20 Modules Jan 19, 2026
SpinnerX added 15 commits January 19, 2026 15:24
SpinnerX
SpinnerX commented Jan 22, 2026
View reviewed changes
Copy link
Member Author

@SpinnerX SpinnerX left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merging this PR. One thing worth noting. The tests on Windows do not execute the tests. As it falsely says all tests passed. When none of the tests are being ran. Will add a GitHub issue related to this topic.

@SpinnerX SpinnerX self-assigned this Jan 22, 2026
@SpinnerX SpinnerX merged commit bfa321f into main Jan 23, 2026
5 checks passed
@SpinnerX SpinnerX deleted the cpp_modules branch January 23, 2026 07:36
@SpinnerX SpinnerX added the improvement improving something to be better label Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@SpinnerX SpinnerX

Labels

improvement improving something to be better

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Moving away from header files to using C++20 modules

2 participants

@SpinnerX