-
Notifications
You must be signed in to change notification settings - Fork 263
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
Added support for All-Tech melee #2986
Added support for All-Tech melee #2986
Conversation
You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-7827634774.zip |
<!-- Patch stats --> | ||
|
||
<li Class="PatchOperationReplace"> | ||
<xpath>Defs/ThingDef[contains(defName, |
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.
This is a rather strange piece of xpath. Why do it this way versus the usual way, if I may ask? Is there something strange in the mod's xml that makes it necessary?
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.
It's a standard xpath, not sure what you mean by strange. Just wanted all of them to have the same cost in stuff.
You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-7868301546.zip |
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.
What's here looks fine, but there's also a Neolithic tech "sharp stick" that needs attended to.
You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-7875734768.zip |
95f4022
into
CombatExtended-Continued:Development
Additions
Describe new functionality added by your code, e.g..
Changes
Added compatibility for All-Tech Melee
References
Mod: [https://steamcommunity.com/sharedfiles/filedetails/?id=2789794530](All-Tech Melee)
Reasoning
It was not compatible and we lack good simple melee weapon mods.
Alternatives
no alternatives considered
Testing
Check tests you have performed: