Skip to content

Conversation

@lpeapnni
Copy link

@lpeapnni lpeapnni commented Jan 5, 2026

About The Pull Request

changes a few lists into alists so the codebase will actually compile for me.

i've tested to make sure that phone contacts aren't fucked up, and they seem to be working fine.
image

Why It's Good For The Game

shit would not even compile for me on 516.1673. i dont think pinning compilation to a specific version of byond is a good idea for the long term.

Changelog

🆑
code: changed lists into alists for phone contact stuff
/:cl:

@lpeapnni lpeapnni requested a review from a team as a code owner January 5, 2026 00:23
@FalloutFalcon FalloutFalcon added this pull request to the merge queue Jan 5, 2026
@XeonMations XeonMations removed this pull request from the merge queue due to a manual request Jan 5, 2026
@XeonMations
Copy link
Collaborator

the latest version we're supposed to be on is 516.1659. Modern TG has issues regarding alists() currently.

@github-actions
Copy link
Contributor

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

@FalloutFalcon
Copy link
Member

Gonna wait for tgstation/tgstation#94522 to be merged or otherwise have TG confirm that alists are stable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants