Skip to content

[WIP] Introduce a skill id enum to make handling more centralized an robust#61

Open
jon-zu wants to merge 1 commit intoiw2d:mainfrom
jon-zu:jz/skill_id_enum
Open

[WIP] Introduce a skill id enum to make handling more centralized an robust#61
jon-zu wants to merge 1 commit intoiw2d:mainfrom
jon-zu:jz/skill_id_enum

Conversation

@jon-zu
Copy link
Contributor

@jon-zu jon-zu commented Oct 16, 2025

This introduces a new package meta and a SkillId enum to attempt to remove most hard coded parts. Just a proof of concept for now.

Some issues may arise:

  1. Temporary Stats reason is rather a multi value, like Items buffs would be invalid Skill Ids
  2. Performance concerns due to conversion via SkillId.fromValue
  3. Some hidden skills are not yet part of the enum

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.

1 participant