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] Refactoring of python files inside cats, cat_relations, and clan_resoucres folders #3221

Closed
wants to merge 42 commits into from

Conversation

filvas2
Copy link
Contributor

@filvas2 filvas2 commented Dec 22, 2024

About The Pull Request

I have carefully gone though all .py files in the following path and tried to make the code cleaner where i deemed it suitable without changing the intent of the code.

  • scripts\cat,
  • scripts\cat_relations
  • scripts\clan_resources

Why This Is Good For ClanGen

I hope it will help with code readability and therefore maintainability

Linked Issues

Proof of Testing

image

I've also tested mediate relationships, kill the leader, break up and make mates of cats and other easily triggered events

Changelog/Credits

refactoring and code cleaning in the following folders:

  • scripts\cat,
  • scripts\cat_relations
  • scripts\clan_resources

Filip Västlund and others added 30 commits December 20, 2024 22:20
@filvas2 filvas2 changed the title [CLEANUP] Refactoring of pythonon files inside cats, cat_relations, and clan_resoucres folders [CLEANUP] Refactoring of python files inside cats, cat_relations, and clan_resoucres folders Dec 22, 2024
@j-gynn j-gynn self-requested a review December 22, 2024 23:17
@filvas2 filvas2 deleted the branch ClanGenOfficial:development December 23, 2024 23:36
@filvas2 filvas2 closed this Dec 23, 2024
@filvas2 filvas2 deleted the development branch December 23, 2024 23:36
@filvas2 filvas2 restored the development branch December 24, 2024 02:57
@filvas2 filvas2 deleted the development branch December 24, 2024 02:58
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.

1 participant