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: Replace lorenevent with skyhanni event #2750

Open
wants to merge 42 commits into
base: beta
Choose a base branch
from

Conversation

NopoTheGamer
Copy link
Contributor

@NopoTheGamer NopoTheGamer commented Oct 17, 2024

Dependencies

What

Gets rid of lorenzevent and uses skyhannievent on everything.
This is an important step of modern version support

Images

image

Changelog Technical Details

  • Replace LorenzEvent with SkyhanniEvent. - nopo & CalMWolfs
    • A few files got changed.

@NopoTheGamer NopoTheGamer changed the title Changes by Cal Backend: Replace lorenevent with skyhanni event Oct 17, 2024
@github-actions github-actions bot added the Backend A backend pull request that will be merged soon label Oct 17, 2024
Copy link

I have detected some issues with your pull request:

Body issues:
Change should start with a capital letter in text: few files got changed.

Title issues:
PR title does not match the expected format of 'Category: Title'

Please fix these issues. For the correct format, refer to the pull request template.

@github-actions github-actions bot added the Wrong Title/Changelog There is an error in the title or changelog label Oct 17, 2024
Copy link

I have detected some issues with your pull request:

Body issues:
Change should start with a capital letter in text: few files got changed.

Please fix these issues. For the correct format, refer to the pull request template.

@github-actions github-actions bot removed the Wrong Title/Changelog There is an error in the title or changelog label Oct 17, 2024
@CalMWolfs CalMWolfs added this to the Version 0.28 milestone Oct 17, 2024
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.

Works for me.

Additionally, when this is merged I will go fix up all prs that this causes to not build or that get merge conflicts because of this

@hannibal002
Copy link
Owner

Too big of an pr. Please don't create that many lines changes at once in the future

@github-actions github-actions bot removed the Backend A backend pull request that will be merged soon label Oct 17, 2024
Copy link

One or more Detekt Failures were detected:

Copy link

One or more Detekt Failures were detected:

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Oct 22, 2024
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/event/spook/TheGreatSpook.kt
#	src/main/java/at/hannibal2/skyhanni/features/misc/TpsCounter.kt
Copy link

Conflicts have been resolved! 🎉

@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Oct 23, 2024
Copy link

One or more Detekt Failures were detected:

Copy link

One or more Detekt Failures were detected:

Copy link

One or more Detekt Failures were detected:

  • ParkourHelper.kt#L45: The function render appears to be too complex based on Cyclomatic Complexity (complexity: 33). Defined complexity threshold for methods is set to '25'

Copy link

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

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Oct 23, 2024
# Conflicts:
#	src/main/java/at/hannibal2/skyhanni/features/misc/LockMouseLook.kt
@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Oct 23, 2024
Copy link

Conflicts have been resolved! 🎉

Copy link

One or more Detekt Failures were detected:

  • ParkourHelper.kt#L45: The function render appears to be too complex based on Cyclomatic Complexity (complexity: 33). Defined complexity threshold for methods is set to '25'

Copy link

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

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Oct 26, 2024
# Conflicts:
#	src/main/java/at/hannibal2/skyhanni/config/ConfigManager.kt
#	src/main/java/at/hannibal2/skyhanni/data/hypixel/chat/PlayerChatManager.kt
#	src/main/java/at/hannibal2/skyhanni/utils/ConfigUtils.kt
#	src/main/java/at/hannibal2/skyhanni/utils/repopatterns/RepoPatternManager.kt
Copy link

Conflicts have been resolved! 🎉

@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Oct 26, 2024
Copy link

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

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Oct 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend A backend pull request that will be merged soon Merge Conflicts There are open merge conflicts with the beta branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants