-
Notifications
You must be signed in to change notification settings - Fork 302
Added sources of gaining rage #1613
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
base: dev
Are you sure you want to change the base?
Changes from 5 commits
14e5cc5
c3a1392
46060ef
ced2e68
2326ae2
4e2cfe7
1191ae5
9e2833d
f67d6fb
061a28b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2707,11 +2707,16 @@ local specialModList = { | |
| ["gain %d+ rage on hit with axes or swords"] = { | ||
| flag("Condition:CanGainRage"), | ||
| }, | ||
| ["gain %d+ rage on melee hit"] = { | ||
| flag("Condition:CanGainRage"), | ||
| }, | ||
| ["gain (%d+) rage on melee hit"] = function(num) return { | ||
| mod("RageOnMeleeHit", "BASE", num), | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should use ModFlag.Melee as a way to only have it work on Melee skills
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. done |
||
| flag("Condition:CanGainRage") | ||
| }end, | ||
| ["gain (%d+) rage when hit by an enemy"] = function(num) return { | ||
| mod("RageWhenHit", "BASE", num), | ||
| flag("Condition:CanGainRage") | ||
| }end, | ||
| ["gain %d+ rage on melee weapon hit"] = { | ||
| flag("Condition:CanGainRage"), | ||
| flag("Condition:CanGainRage") | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can use the weapon modflag for this similar to how we handle other weapon hit mods
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I actually deleted that parse since it seemed like a duplicate to me, the nodes still work fine without that. |
||
| }, | ||
| ["gain %d+ rage on ([%D]+)"] = { | ||
| flag("Condition:CanGainRage"), | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should change to be Gain on Hit so it can work for other sources
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done