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

TUNIC: Change Tunic to TUNIC #2720

Merged
merged 5 commits into from
Jan 18, 2024
Merged

Conversation

ReverM
Copy link
Contributor

@ReverM ReverM commented Jan 14, 2024

Changed init.py so that the name of the game appears as TUNIC instead of Tunic for uniformity.

How was this tested?

Ran WebHost locally

@black-sliver
Copy link
Member

This change would break existing yamls. So unless @silent-destroyer want's to absolutely do this, I would not recommend it.

@ReverM
Copy link
Contributor Author

ReverM commented Jan 14, 2024

I have discussed this change with silent beforehand

@alwaysintreble
Copy link
Collaborator

not only does this break the yamls but it also requires a client change

@alwaysintreble alwaysintreble added the is: refactor/cleanup Improvements to code/output readability or organizization. label Jan 14, 2024
@ReverM
Copy link
Contributor Author

ReverM commented Jan 14, 2024

not only does this break the yamls but it also requires a client change

This also has been discussed with Silent

@black-sliver
Copy link
Member

I would like them to use the github feature to approve this PR then.

@Berserker66 Berserker66 merged commit 4c901dc into ArchipelagoMW:main Jan 18, 2024
12 checks passed
@ReverM ReverM deleted the tunic-to-TUNIC branch January 18, 2024 01:38
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: refactor/cleanup Improvements to code/output readability or organizization.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants