Skip to content

Feature: Fire Freeze Additions - #5224

Merged
hannibal002 merged 13 commits into
hannibal002:betafrom
Fazfoxy:More-Empa-Things
Feb 26, 2026
Merged

hannibal002 merged 13 commits into
hannibal002:betafrom
Fazfoxy:More-Empa-Things

Conversation

@Fazfoxy

@Fazfoxy Fazfoxy commented Feb 9, 2026 •

Copy link
Copy Markdown
Contributor

Dependencies

What

This is Another Empa Feature Port thing, code might be fucked from taking from 1.8 manually

Images image image

Changelog New Features

  • Added Fire Freeze Features. - Empa & Fazfoxy
    • Timer/Box on Mobs for Freeze Timer.
    • Timer until Freeze Activation.
    • Particle Replacer with Circular Line.

Comment thread src/main/java/at/hannibal2/skyhanni/features/combat/FireFreezeFeatures.kt Outdated
@github-actions github-actions Bot added the Wrong Title/Changelog There is an error in the title or changelog label Feb 9, 2026
@github-actions

github-actions Bot commented Feb 9, 2026

Copy link
Copy Markdown

I have detected some issues with your pull request:

Body issues:
Change should end with a full stop in text: Fire Freeze Features

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

@github-actions github-actions Bot added Detekt Has detekt problem and removed Wrong Title/Changelog There is an error in the title or changelog labels Feb 9, 2026
@github-actions

github-actions Bot commented Feb 9, 2026

Copy link
Copy Markdown

2 Detekt Failures

Rules flagged (2): MaxLineLength, UnusedPrivateProperty
Files flagged (2): WorldRenderUtils.kt, FireFreezeFeatures.kt

@github-actions github-actions Bot removed the Detekt Has detekt problem label Feb 9, 2026

@hannibal002 hannibal002 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Overall the feature looks cool and well coded!

Comment thread src/main/java/at/hannibal2/skyhanni/config/features/combat/CombatConfig.kt Outdated
Comment thread src/main/java/at/hannibal2/skyhanni/config/features/combat/FireFreezeConfig.kt Outdated
Comment thread src/main/java/at/hannibal2/skyhanni/features/combat/FireFreezeFeatures.kt Outdated
Comment thread src/main/java/at/hannibal2/skyhanni/features/combat/FireFreezeFeatures.kt Outdated
Comment thread src/main/java/at/hannibal2/skyhanni/config/features/combat/FireFreezeConfig.kt Outdated
Comment thread src/main/java/at/hannibal2/skyhanni/config/features/combat/FireFreezeConfig.kt Outdated
Comment thread src/main/java/at/hannibal2/skyhanni/features/combat/FireFreezeFeatures.kt Outdated
Comment thread src/main/java/at/hannibal2/skyhanni/features/combat/FireFreezeFeatures.kt Outdated
Comment thread src/main/java/at/hannibal2/skyhanni/features/combat/FireFreezeFeatures.kt Outdated
@github-actions github-actions Bot added the Detekt Has detekt problem label Feb 9, 2026
@github-actions

github-actions Bot commented Feb 9, 2026

Copy link
Copy Markdown

10 Detekt Failures

Rules flagged (5): AvoidBritishSpelling, NoUnusedImports, Wrapping, SpacingAroundComma, MaxLineLength
Files flagged (3): FireFreezeConfig.kt, ItemAbilityConfig.kt, FireFreezeFeatures.kt

Rule violations

@github-actions github-actions Bot removed the Detekt Has detekt problem label Feb 9, 2026

@hannibal002 hannibal002 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

logic looks fire.

only typos.

btw, please add the prefix "Add ..." to the chagnelog line for new features

Comment thread src/main/java/at/hannibal2/skyhanni/features/itemabilities/FireFreezeFeatures.kt Outdated
Comment thread src/main/java/at/hannibal2/skyhanni/features/itemabilities/FireFreezeFeatures.kt Outdated

@hannibal002 hannibal002 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM

@hannibal002
hannibal002 merged commit 9d8abff into hannibal002:beta Feb 26, 2026
9 checks passed
@Fazfoxy
Fazfoxy deleted the More-Empa-Things branch March 2, 2026 20:06
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.

3 participants