Skip to content
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

Merged

Conversation

databobek
Copy link
Contributor

Additions

Describe new functionality added by your code, e.g..

  • Compatibility for All-Tech Melee

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:

  • Compiles without warnings
  • Game runs without errors
  • (For compatibility patches) ...with and without patched mod loaded
  • Playtested a colony (25 ingame years)

@databobek databobek requested review from a team as code owners February 8, 2024 09:42
Copy link

github-actions bot commented Feb 8, 2024

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-7827634774.zip

@github-actions github-actions bot added the Download in Comments This PR has a zipfile download available. label Feb 8, 2024
<!-- Patch stats -->

<li Class="PatchOperationReplace">
<xpath>Defs/ThingDef[contains(defName,
Copy link
Contributor

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?

Copy link
Contributor Author

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.

Copy link

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-7868301546.zip

Copy link
Contributor

@N7Huntsman N7Huntsman left a 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.

Copy link

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-7875734768.zip

@N7Huntsman N7Huntsman merged commit 95f4022 into CombatExtended-Continued:Development Feb 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Download in Comments This PR has a zipfile download available.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants