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

Reset best kills, items, secrets on new highest skill completion #629

Merged
merged 1 commit into from
Mar 27, 2025

Conversation

thecnoNSMB
Copy link
Contributor

I noticed while reading the source code that these are not reset when they probably should be. I reset them to 0 since that appears to be the value they initialize with. Creating a PR as requested on the Discord. (I'm not a C developer, please have patience, thanks!)

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@thecnoNSMB thecnoNSMB marked this pull request as draft March 17, 2025 17:38
@thecnoNSMB
Copy link
Contributor Author

thecnoNSMB commented Mar 17, 2025

Converting this to a draft since I would like the changes to be reviewed by an actual C developer. (I'm not sure if that's what drafts are for, feel free to change it back if you see fit.) (yeah i'm pretty sure that's not what drafts are for, nevermind!)

@thecnoNSMB thecnoNSMB marked this pull request as ready for review March 17, 2025 17:41
@Pedro-Beirao Pedro-Beirao merged commit a88610c into kraflab:master Mar 27, 2025
5 checks passed
@Pedro-Beirao
Copy link
Collaborator

Thank you!

For clarification, this fixes a case where you first complete a level in a lower skill with many kills/secrets/items, but when you complete it again in an higher skill but less kill/secrets/items it would preserve the higher stats from the lower skill

@thecnoNSMB thecnoNSMB deleted the patch-1 branch March 29, 2025 07:35
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

2 participants