Skip to content
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.

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

Split Math and Interop out of OpenLoco #1771

Merged
merged 4 commits into from Jan 29, 2023

Conversation

ZehMatt
Copy link
Contributor

@ZehMatt ZehMatt commented Jan 27, 2023

No description provided.

@ZehMatt ZehMatt force-pushed the cmake-splits branch 2 times, most recently from 8ea5452 to 0ac11af Compare January 27, 2023 19:25
Copy link
Contributor

@duncanspumpkin duncanspumpkin left a comment

Choose a reason for hiding this comment

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

All looks sensible

Copy link
Member

@AaronVanGeffen AaronVanGeffen left a comment

Choose a reason for hiding this comment

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

More modularity sounds like a good way forward. Haven't been able to test these changes, but no objections from me.

@duncanspumpkin duncanspumpkin merged commit f653e2d into OpenLoco:master Jan 29, 2023
@duncanspumpkin duncanspumpkin added this to the v23.01+ milestone Jan 29, 2023
@ZehMatt ZehMatt deleted the cmake-splits branch January 29, 2023 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants