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

Atrocity ships #293

Open
wants to merge 302 commits into
base: ES-Unstable-Content-XXXI
Choose a base branch
from

Conversation

kestrel1110
Copy link
Owner

NOTICE: Delete the sections that do not apply to your PR, and fill out the section that does.
(You can open a PR to add or improve a section, if you find them lacking!)


Content (Artwork / Missions / Jobs)

Summary

{{summarize your content! Include links to related issues, in-game screenshots, etc.}}

Save File

This save file can be used to play through the new mission content:
{{attach a save file that allows people to easily test your added mission content or see your new in-game art}}

Artwork Checklist

  • I updated the copyright attributions, or decline to claim copyright of any assets produced or modified
  • I created a PR to the endless-sky-assets repo with the necessary image, blend, and texture assets: {{insert PR link}}
  • I created a PR to the endless-sky-high-dpi repo with the @2x versions of these art assets: {{insert PR link}}

Bugfix: This PR addresses issue #{{insert number}}

Fix Details

{{add details}}

Testing Done

{{describe how you tested that the fix doesn't introduce other issues}}

Save File

This save file can be used to verify the bugfix. The bug will occur when using {{insert commit hash / version}}, and will not occur when using this branch's build.
{{attach a save file that can be used to verify your bugfix. It MUST have no plugin requirements}}


Feature: This PR implements the feature request detailed and discussed in issue #{{insert number}}

Feature Details

{{add details about the feature you implemented}}

UI Screenshots

{{attach before + after screenshots of any changes to UI, or replace this line with "N/A"}}

Usage Examples

{{if this feature is used in the data files, provide examples!}}

Testing Done

{{describe how you tested the new feature}}

Automated Tests Added

{{describe the automated tests you added (using the unit-test framework, integration-test framework or otherwise) to reduce the risk of regressions in the future. "N/A" if this PR is not for an in-game feature that needs to remain functional in the future. }}

Performance Impact

{{describe any performance impact (positive or negative). "N/A" if no performance-critical code is changed. }}

TomGoodIdea and others added 30 commits June 8, 2023 18:19
…ns (endless-sky#8860)

* An escort with the derelict personality is in the same system as the player's flagship, and has run out of fuel will no longer fly outward forever.
* A ship that is not an escort, has nothing to do, and won't leave the system, in a system with no StellarObject, will no longer fly outward forever.
* A ship will now de-target a friendly target that has left the system.
* A ship that has decided to de-target a friendly target will complete AI decisions further in that step instead of stopping at that point.
* Ships that want to de-target a friendly target will now actually clear the ships target pointer instead of clearing a local pointer.
* Ships that are being assisted can no longer choose to assist someone else.
* Ships with certain special behaviors (like surveillance or mining) will now do that if they are stranded without anyone to ask for help. Previously, they would become locked in a "do nothing" state.
… to cloak (endless-sky#8874)

* Also, add the new personality to the Starling the player needs to scan in the Remnant: Keystone Research missions.
* Also, refactor AI::DoCloak slightly to improve readability (warp-core).

Co-authored-by: warp-core <[email protected]>
…ky#8892)

In some cases an escort decides to regain its fuel using its ramscoop instead of landing on a suitable planet, which is faster.
…lanets with the "remnant station" attribute as destinations (endless-sky#8897)
When creating a release the steam depots are now packaged and uploaded as a workflow artifact. They can be uploaded directly to Steam.
Co-authored-by: tibetiroka <[email protected]>
Co-authored-by: warp-core <[email protected]>
TomGoodIdea and others added 29 commits August 30, 2023 08:31
Improved the planet label code to allow for better placement (without overlaps) of planet labels.
…royed (but not captured) ships (endless-sky#9264)

Changed the "on kill" action behavior to match the "kill" NPC objective, triggering on both ship destruction and capture
* update

* don't show progress

* remove default value
Co-authored-by: Amazinite <[email protected]>
@TomGoodIdea TomGoodIdea deleted the atrocity-ships branch February 3, 2024 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.