Skip to content

Conversation

Maleclypse
Copy link
Member

Summary

None

Purpose of change

Fixes the issues where Gracken parts were broken as unusable comestibles. Might make a test to confirm these work in perpetuity.

Describe the solution

Make them single use tools and add code to make Type conflicting traits get deleted when an EOC adds a trait with the same type.

Describe alternatives you've considered

Deleting gracken characters entirely if I couldn't get this to work as the alternatives were things that would spiral out of control.

Testing

Tested and it works.

Additional context

@github-actions github-actions bot added NPC / Factions NPCs, AI, Speech, Factions, Ownership [JSON] Changes (can be) made in JSON Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies [C++] Changes (can be) made in C++. Previously named `Code` Mods: Xedra Evolved Anything to do with Xedra Evolved labels Aug 31, 2025
@Maleclypse Maleclypse marked this pull request as draft August 31, 2025 21:54
@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label Aug 31, 2025
@Maleclypse
Copy link
Member Author

I built this on top of #82649 for some reason that made sense when I started but no longer does. Needs #82649 to merge first.

@Maleclypse Maleclypse marked this pull request as ready for review August 31, 2025 22:10
@Maleclypse Maleclypse changed the title [ Xedra Evolved ] Fix Gracken trait improving items Requires another PR to merge first or me to interactively rebase [ Xedra Evolved ] Fix Gracken trait improving items Aug 31, 2025
@github-actions github-actions bot added astyled astyled PR, label is assigned by github actions Code: Tests Measurement, self-control, statistics, balancing. and removed astyled astyled PR, label is assigned by github actions labels Aug 31, 2025
@github-actions github-actions bot added astyled astyled PR, label is assigned by github actions and removed astyled astyled PR, label is assigned by github actions labels Sep 1, 2025
Maleclypse and others added 5 commits September 2, 2025 23:07
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@Maleclypse Maleclypse force-pushed the Gracken-get-their-crap-working branch from 42f9184 to 5a261e1 Compare September 3, 2025 04:08
@github-actions github-actions bot added Crafting / Construction / Recipes Includes: Uncrafting / Disassembling and removed astyled astyled PR, label is assigned by github actions labels Sep 3, 2025
@github-actions github-actions bot added the astyled astyled PR, label is assigned by github actions label Sep 3, 2025
@Maleclypse Maleclypse changed the title Requires another PR to merge first or me to interactively rebase [ Xedra Evolved ] Fix Gracken trait improving items [ Xedra Evolved ] Fix Gracken trait improving items Sep 3, 2025
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Sep 13, 2025
@GuardianDll
Copy link
Member

Clang error should be resolved

@GuardianDll GuardianDll merged commit 35b6c4e into CleverRaven:master Sep 22, 2025
37 of 39 checks passed
Blueflowerss pushed a commit to Blueflowerss/Cataclysm-DDA that referenced this pull request Sep 22, 2025
* Update monster_drops.json

* Quick and dirty craft failure explosions

code changes

* Update crafting.cpp

* Update mutation.cpp

* Update src/mutation.cpp

* Update src/mutation.cpp

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update mutation.cpp

* Update mutation.cpp

* Apply suggestions from code review

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Apply suggestions from code review

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update src/mutation.cpp

* Apply suggestions from code review

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* items work again

* Fix EOC not respecting mutation type

* Test

* Update src/npctalk.cpp

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update npctalk.cpp

* Apply suggestions from code review

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update src/npctalk.cpp

* Update src/npctalk.cpp

* Update src/npctalk.cpp

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update iuse_test.cpp

* Apply suggestions from code review

* Update iuse_test.cpp

* Update src/mutation.cpp

* Update tests/iuse_test.cpp

* Apply suggestions from code review

* Update npctalk.cpp

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@Maleclypse Maleclypse deleted the Gracken-get-their-crap-working branch September 22, 2025 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` Code: Tests Measurement, self-control, statistics, balancing. Crafting / Construction / Recipes Includes: Uncrafting / Disassembling [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Mods: Xedra Evolved Anything to do with Xedra Evolved Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies NPC / Factions NPCs, AI, Speech, Factions, Ownership
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants