Closed GuyPaddock closed 3 months ago
[!IMPORTANT]
Auto Review Skipped
More than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review.
172 files out of 297 files are above the max files limit of 50. Please upgrade to Pro plan to get higher limits.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
This mega PR reorganizes the project and implements a ton of functionality related to melee attack abilities, inventory, and damage calculations.
This PR does NOT include logic required for ranged attacks, including attacks that have ammunition.
Highlights
CharacterAttributeStatics
into "source" and "target" variants, since GAs/GEs will need access to detail from both.Let
andBeforeAll
capabilities from Rspec. The version in this project is a cleaned up and refactored version of what's in UE 5.3, but with additional functionality, including fixing the way stack traces in specs are linked to from the Session Frontend. Epic did not design the original version to be extensible. Several existing tests in this project have been rewritten to use the new functionality..spec
suffix to all functional tests.