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

Backend: EnumMap and Storage Cleanup #3200

Merged
merged 8 commits into from
Jan 19, 2025

Conversation

ItsEmpa
Copy link
Contributor

@ItsEmpa ItsEmpa commented Jan 10, 2025

What

This PR adds some helper methods for creating new EnumMaps, and also modifies profile and player storage to use them, and change some other stuff in it as well.

Changelog Technical Details

  • Added EnumMap helper methods. - Empa

@github-actions github-actions bot added Backend A backend pull request that will be merged soon Detekt Has detekt problem labels Jan 10, 2025
Copy link

One or more Detekt Failures were detected:

Copy link
Contributor

@DavidArthurCole DavidArthurCole left a comment

Choose a reason for hiding this comment

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

Also object StaticDurations in TimeUtils can get nuked.

@hannibal002 hannibal002 added this to the Version 2.0.0 milestone Jan 10, 2025
@hannibal002 hannibal002 added the Soon This Pull Request will be merged within the next couple of betas label Jan 10, 2025
Copy link

One or more Detekt Failures were detected:

Signed-off-by: Empa <[email protected]>
@github-actions github-actions bot added Merge Conflicts There are open merge conflicts with the beta branch. and removed Detekt Has detekt problem labels Jan 11, 2025
Copy link

This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes.

# Conflicts:
#	src/main/java/at/hannibal2/skyhanni/features/mining/glacitemineshaft/CorpseTracker.kt
@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Jan 12, 2025
Copy link

Conflicts have been resolved! 🎉

@github-actions github-actions bot added the Detekt Has detekt problem label Jan 12, 2025
Copy link

One or more Detekt Failures were detected:

@github-actions github-actions bot removed the Detekt Has detekt problem label Jan 12, 2025
Copy link
Collaborator

@CalMWolfs CalMWolfs left a comment

Choose a reason for hiding this comment

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

code lgtm

@hannibal002 hannibal002 merged commit a01c042 into hannibal002:beta Jan 19, 2025
6 checks passed
@github-actions github-actions bot removed Soon This Pull Request will be merged within the next couple of betas Backend A backend pull request that will be merged soon labels Jan 19, 2025
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.

4 participants