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

[CLEANUP] Renaming Stuff #2436 #2713

Draft
wants to merge 13 commits into
base: development
Choose a base branch
from

Conversation

PoppyBlossom
Copy link
Contributor

@PoppyBlossom PoppyBlossom commented Sep 21, 2024

About The Pull Request

This PR contains some name changes suggested by #2436

  • gender -> sex (to remove confusion around what this is)
  • ID -> cat_id (should be lowercase and id is taken by a Python builtin)
  • favourite -> favorite (Murica) [CLEANUP] Cat config files moved to, surprise, CatConfig #2437 (Done Already)
  • mate -> current_mates (in line with current_apprentices and the poly update)
  • previous_mates -> former_mates (in line with former_apprentices and former_mentor)

Part of: #2436

j-gynn mentioned that they don't need help with their project but renaming stuff is really time-consuming, so I guess I can help a bit?...

@PoppyBlossom PoppyBlossom changed the title Remamed Gender & "GenderAlign" Remamed Gender & "GenderAlign" #2436 Sep 21, 2024
@PoppyBlossom PoppyBlossom changed the title Remamed Gender & "GenderAlign" #2436 Renaming Stuffs #2436 Sep 21, 2024
@PoppyBlossom PoppyBlossom marked this pull request as ready for review September 21, 2024 03:11
@PoppyBlossom PoppyBlossom changed the title Renaming Stuffs #2436 Renaming Stuff #2436 Sep 21, 2024
@j-gynn
Copy link
Contributor

j-gynn commented Sep 21, 2024

Hey, thanks a bunch (/gen) for helping! I think the senior devs may close this PR as we're currently in feature-freeze (only accepted PRs right now are bugfixes and art/writing content). However! Very grateful for the help.

Are you in the Discord server? Would love to have a chat and see if some of the parts of this can be formally "allocated" to you to save me doing them too by mistake 😂 You'd also get access to beta testers, which are almost essential for major internal changes like this one to make sure we don't miss anything (ask me how I know. I definitely didn't corrupt a bunch of people's saves with my first ever pushed PR to this repo, nope... /s).

Also, just doing some testing - since this modifies the JSON save terms it causes a fatal bug when you try and load a save made from this branch. For now, I'd suggest reverting the JSON changes - once all the desired changes to cats.py have been made, then we can go through and make a new version of the JSON and update the save converter to make all those changes.

@PoppyBlossom
Copy link
Contributor Author

Thank you so much for the advice! I am not currently on the Discord server, but I will try to join as soon as possible.

@PoppyBlossom PoppyBlossom marked this pull request as draft September 21, 2024 15:23
@PoppyBlossom PoppyBlossom changed the title Renaming Stuff #2436 [CLEANUP] Renaming Stuff #2436 Sep 21, 2024
@PoppyBlossom PoppyBlossom deleted the Remaming-Stuff-#2436 branch September 29, 2024 03:14
@PoppyBlossom PoppyBlossom restored the Remaming-Stuff-#2436 branch September 29, 2024 03:17
@PoppyBlossom PoppyBlossom deleted the Remaming-Stuff-#2436 branch September 29, 2024 03:22
@PoppyBlossom PoppyBlossom restored the Remaming-Stuff-#2436 branch September 29, 2024 03:28
@PoppyBlossom PoppyBlossom reopened this Sep 29, 2024
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.

2 participants