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.

Sign up for GitHub

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

Jump to bottom

Minor turret research project location tweaks #3158

Merged
merged 1 commit into from
May 31, 2024
Merged

Minor turret research project location tweaks #3158

merged 1 commit into from
May 31, 2024

Conversation

SamaelGray
Copy link
Contributor

Changes

  • Moved CE's heavy auto turret research project slightly up to be on the same Y level as the vanilla autocannon turret.
  • Removed the Y axis patch of the vanilla autocannon turret.

Reasoning

  • Alignment issues. CE's heavy auto turret tech now sits besides vanilla's AC turret tech and does not stick to the precision rifling project. AC turret tech now has the 3.1 Y coordinate by default making the patch redundant.

Testing

Check tests you have performed:

  • Compiles without warnings
  • Game runs without errors

@SamaelGray SamaelGray requested review from a team as code owners May 28, 2024 23:02
@github-actions GitHub Actions
Copy link

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

@github-actions github-actions bot added the Download in Comments This PR has a zipfile download available. label May 28, 2024
@N7Huntsman N7Huntsman merged commit 9898616 into CombatExtended-Continued:Development May 31, 2024
2 checks passed
@SamaelGray SamaelGray deleted the SamaelGray-TechTreeTweaks branch May 31, 2024 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@N7Huntsman N7Huntsman N7Huntsman approved these changes

Assignees
No one assigned
Labels
Download in Comments This PR has a zipfile download available.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SamaelGray @N7Huntsman