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

Finish updating copyleft headers - 0.9.x #1030

Open
wants to merge 3 commits into
base: 0.9.x
Choose a base branch
from

Conversation

stephengtuggy
Copy link
Contributor

@stephengtuggy stephengtuggy commented Feb 14, 2025

Code Changes:

Issues:

  • N/A

Purpose:

  • What is this pull request trying to do? Finish the copyleft header updates for the game engine, 0.9.x branch. Included are 460 .cpp files and 2 .h files.
  • What release is this for? 0.9.x
  • Is there a project or milestone we should apply this to? 0.9.x

@stephengtuggy stephengtuggy added this to the 0.9.x milestone Feb 14, 2025
@stephengtuggy stephengtuggy self-assigned this Feb 14, 2025
@stephengtuggy stephengtuggy changed the title Update copyleft headers in 460 remaining .cpp files; touch up a coupl… Finish updating copyleft headers - 0.9.x Feb 14, 2025
Copy link
Member

Choose a reason for hiding this comment

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

@royfalk should check this file

Copy link
Member

@BenjamenMeyer BenjamenMeyer left a comment

Choose a reason for hiding this comment

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

for the most part LGTM
though there's a few more things than just header changes, namely some type-casting which @royfalk should verify; and we should get our testers to test this one (@evertvorster @kheckwrecker )

@BenjamenMeyer BenjamenMeyer mentioned this pull request Feb 17, 2025
2 tasks
@kheckwrecker
Copy link

Built 0.9x instance with this PR and PR#1035 merged in. Did not see any particular issues with this PR.

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.

3 participants