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

Refactors Xeno name generation and set hive and update #4992

Merged
merged 2 commits into from
Dec 1, 2023

Conversation

Birdtalon
Copy link
Contributor

@Birdtalon Birdtalon commented Nov 21, 2023

About the pull request

Refactors xenomorph name generation. Everything is now handled in generate_name(). Refactored out istype(src)
Larva name generation no longer handled in update_icons()

All hive datums now have a list of 1-999 which gets picked from. If that list is empty it will fall-back on 0. I can add functionality for generating a new list but seriously when do we ever get through 999 lives.

Removes istype(src) from set_hive_and_update()

Adds "Infernal" name prefix for xeno who rolls 666 as a little easter egg.

Explain why it's good for the game

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
add: "Infernal" name prefix for xeno who rolls number 666
code: Refactors xeno name generation. Larva name generation and removes some istype(src)
/:cl:

@github-actions github-actions bot added Feature Feature coder badge Code Improvement Make the code longer labels Nov 21, 2023
Copy link
Contributor

github-actions bot commented Dec 1, 2023

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label Dec 1, 2023
@fira fira added this pull request to the merge queue Dec 1, 2023
Merged via the queue into cmss13-devs:master with commit 354e9b8 Dec 1, 2023
26 checks passed
cm13-github added a commit that referenced this pull request Dec 1, 2023
github-merge-queue bot pushed a commit that referenced this pull request Dec 3, 2023
# About the pull request

#4992 broke non normal hive nick numbering due to an oversight. You will
laugh when you see why.
This fixes that.

# Explain why it's good for the game
# Testing Photographs and Procedure
<details>
<summary>Screenshots & Videos</summary>

Put screenshots and videos here with an empty line between the
screenshots and the `<details>` tags.

</details>


# Changelog
🆑 
fix: Fixed non normal xeno hive nick numbering
/🆑
@Birdtalon Birdtalon deleted the xenosethive branch December 21, 2023 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Improvement Make the code longer Feature Feature coder badge Stale beg a maintainer to review your PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants