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

Improvement: Translate SkillUpgradeRecommendation to Kotlin #440

Merged
merged 4 commits into from
Jul 24, 2024

Conversation

j10a1n15
Copy link
Collaborator

Now the only files left are dungeon party stuff

Signed-off-by: J10a1n15 <45315647+j10a1n15@users.noreply.github.com>
@j10a1n15 j10a1n15 requested a review from Su386yt July 22, 2024 20:04
j10a1n15 added 2 commits July 22, 2024 22:04
Signed-off-by: J10a1n15 <45315647+j10a1n15@users.noreply.github.com>
Signed-off-by: J10a1n15 <45315647+j10a1n15@users.noreply.github.com>
Copy link
Member

@Su386yt Su386yt left a comment

Choose a reason for hiding this comment

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

This needs a complete rewrite more than anything but I'll approve after this change

…skills/SkillUpgradeRecommendation.kt

Co-authored-by: Su386 <83100266+Su386yt@users.noreply.github.com>
Signed-off-by: J10a1n15 <45315647+j10a1n15@users.noreply.github.com>
@j10a1n15
Copy link
Collaborator Author

This needs a complete rewrite more than anything but I'll approve after this change

Tried that twice and failed both times with the calculations
They were never the same as the previous ones

@j10a1n15 j10a1n15 requested a review from Su386yt July 24, 2024 15:09
@Su386yt
Copy link
Member

Su386yt commented Jul 24, 2024

I mean the logic should be rewritten haha

@j10a1n15
Copy link
Collaborator Author

I mean the logic should be rewritten haha

yeah thats what I tried to do

@Su386yt
Copy link
Member

Su386yt commented Jul 24, 2024

If the logic needs to be rewritten, then it doesn't really matter if they match or not

@Su386yt Su386yt merged commit 63bbbe7 into PartlySaneStudios:main Jul 24, 2024
2 checks passed
@j10a1n15 j10a1n15 deleted the improvement/su-kotlin branch July 24, 2024 15:13
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.

2 participants