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

Ruin map template tags #3077

Merged
merged 11 commits into from
Sep 20, 2024
Merged

Ruin map template tags #3077

merged 11 commits into from
Sep 20, 2024

Conversation

FalloutFalcon
Copy link
Member

About The Pull Request

turns all the stuff in map_catalogue.txt into proper defines that can be used soon

Why It's Good For The Game

this should come in useful for spawning ruins of certain qualities, scanning a planet for information about them and other use cases.

Changelog

🆑
code: added ruin tags to ruin map templates to be used soon for stuff.
/:cl:

@github-actions github-actions bot added Map Change Tile placing is hard. Thank you for your service. Code change Watch something violently break. Merge Conflict Use Git Hooks, you're welcome. labels Jun 6, 2024
Copy link
Contributor

github-actions bot commented Jun 6, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Jun 26, 2024
@FalloutFalcon FalloutFalcon marked this pull request as ready for review June 26, 2024 16:01
@FalloutFalcon FalloutFalcon requested a review from a team as a code owner June 26, 2024 16:01
@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Jul 23, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Aug 10, 2024
@rye-rice
Copy link
Member

rye-rice commented Sep 9, 2024

@shiptest-ss13/maptainers

Copy link
Contributor

@meemofcourse meemofcourse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i would wait on @Latentish but this seems ok

@FalloutFalcon
Copy link
Member Author

This is gonna need a quick follow up pr prob done by a maptainer since some stuff grew slightly out of date since this pr has been open for a while.
The new maps will need to be converted to proper tags

@rye-rice
Copy link
Member

@Latentish

Copy link
Member

@Latentish Latentish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@rye-rice rye-rice added this pull request to the merge queue Sep 20, 2024
Merged via the queue into shiptest-ss13:master with commit 18631e7 Sep 20, 2024
14 checks passed
MysticalFaceLesS pushed a commit to CeladonSS13/Shiptest that referenced this pull request Sep 22, 2024
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->

## About The Pull Request
turns all the stuff in map_catalogue.txt into proper defines that can be
used soon

<!-- Describe The Pull Request. Please be sure every change is
documented or this can delay review and even discourage maintainers from
merging your PR! -->

## Why It's Good For The Game
this should come in useful for spawning ruins of certain qualities,
scanning a planet for information about them and other use cases.

<!-- Please add a short description of why you think these changes would
benefit the game. If you can't justify it in words, it might not be
worth adding. -->

## Changelog

:cl:
code: added ruin tags to ruin map templates to be used soon for stuff.
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->
MrCat15352 pushed a commit to MrCat15352/MrCat that referenced this pull request Dec 24, 2024
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->
turns all the stuff in map_catalogue.txt into proper defines that can be
used soon

<!-- Describe The Pull Request. Please be sure every change is
documented or this can delay review and even discourage maintainers from
merging your PR! -->
this should come in useful for spawning ruins of certain qualities,
scanning a planet for information about them and other use cases.

<!-- Please add a short description of why you think these changes would
benefit the game. If you can't justify it in words, it might not be
worth adding. -->

:cl:
code: added ruin tags to ruin map templates to be used soon for stuff.
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break. Map Change Tile placing is hard. Thank you for your service.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants