Skip to content

Adjust Atalui Bloodletting cap handling#3

Merged
eriksegecs merged 2 commits into
devfrom
codex/edit-atalui-s-bloodletting-lineage-support-008r4x
Dec 7, 2025
Merged

Adjust Atalui Bloodletting cap handling#3
eriksegecs merged 2 commits into
devfrom
codex/edit-atalui-s-bloodletting-lineage-support-008r4x

Conversation

@eriksegecs
Copy link
Copy Markdown
Owner

Summary

  • raise the per-stat limit to cap Atalui's Bloodletting physical damage gain at 40%
  • mirror the updated limit in exported skill data

Testing

  • not run (not requested)

Codex Task

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +534 to +535
levels = {
[1] = { levelRequirement = 0, },
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Keep Atalui life cost multiplier

The levels block for SupportAtaluiBloodlettingPlayer now only sets levelRequirement, dropping the previous manaMultiplier = 50. Without that multiplier the support falls back to the default 100% cost, so every build using Atalui’s Bloodletting will have its life cost (and the resulting “per Life cost” physical damage gain) reduced by a third compared to the pre-change values. This change isn’t mentioned in the cap update and makes the skill’s cost and damage scaling diverge from the prior implementation—please retain the 50% cost multiplier when adding the 40% cap.

Useful? React with 👍 / 👎.

@eriksegecs eriksegecs merged commit 84ce5f5 into dev Dec 7, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant