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

Xeno deaths are now tracked in the DB #8257

Merged
merged 10 commits into from
Jan 30, 2025

Conversation

Zonespace27
Copy link
Contributor

@Zonespace27 Zonespace27 commented Jan 26, 2025

About the pull request

Tracks a ton of data relating to all xeno deaths
Additionally fixes xenos not being logged in round_stats.log files

Explain why it's good for the game

Partially helping me learn how to navigate this DB API while also allowing for me to cook some stat tracking stuff on the backend

Changelog

🆑
admin: Xenomorphs are now properly logged in round_stats.log files.
/:cl:

@cmss13-ci cmss13-ci bot added the Missing Changelog Maintainers always document their changes. label Jan 26, 2025
@Zonespace27 Zonespace27 marked this pull request as draft January 26, 2025 20:38
@Drulikar
Copy link
Contributor

Can you also tweak whatever has made it so all the xeno values that get output to round_stats.log are 0?

@Zonespace27
Copy link
Contributor Author

Can you also tweak whatever has made it so all the xeno values that get output to round_stats.log are 0?

Fixed

@Zonespace27 Zonespace27 marked this pull request as ready for review January 26, 2025 21:24
@cmss13-ci cmss13-ci bot added Admin Make things harder for admins and removed Missing Changelog Maintainers always document their changes. labels Jan 26, 2025
@Zonespace27 Zonespace27 marked this pull request as draft January 27, 2025 02:24
@Zonespace27 Zonespace27 marked this pull request as ready for review January 28, 2025 03:51
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Jan 28, 2025
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Jan 29, 2025
@harryob harryob added this pull request to the merge queue Jan 30, 2025
Merged via the queue into cmss13-devs:master with commit 63ab11d Jan 30, 2025
27 of 28 checks passed
cmss13-ci bot added a commit that referenced this pull request Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin Make things harder for admins
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

4 participants