Skip to content

Conversation

@KANAjetzt
Copy link
Member

@KANAjetzt KANAjetzt commented Apr 18, 2023

Move mod_data from ModLoader to ModLoaderStore

This moves mod_data from mod_loader.gd to mod_loader_store.gd, enabling easier access from different classes.

Breaking

  • Two mods in my list, because they directly access ModLoader.mod_data.
    if ModLoader.mod_data.keys().has("damiBenro-Junker")
  • The default mod menu in Brotato for the same reason.
    for key in ModLoader.mod_data:

Opened #219 and #221 to prevent this in the future.




works towards #153

This moves `mod_data` from *mod_loader.gd* to *mod_loader_store.gd*, enabling easier access from different classes.
@KANAjetzt KANAjetzt added the refactor / cleanup Improves readability or maintainability label Apr 18, 2023
@KANAjetzt KANAjetzt added this to the v6.0.0 milestone Apr 18, 2023
@KANAjetzt KANAjetzt requested review from a team, Qubus0, ithinkandicode and otDan April 18, 2023 20:08
@KANAjetzt KANAjetzt self-assigned this Apr 18, 2023
@KANAjetzt KANAjetzt added the breaking Breaking change label Apr 18, 2023
Copy link
Collaborator

@Qubus0 Qubus0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good 👍

@KANAjetzt KANAjetzt added this pull request to the merge queue Apr 21, 2023
Merged via the queue into GodotModding:development with commit bec0f52 Apr 21, 2023
@KANAjetzt KANAjetzt deleted the refactor_move_mod_data_to_ModLoaderStore branch April 21, 2023 06:54
@Qubus0 Qubus0 mentioned this pull request Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Breaking change refactor / cleanup Improves readability or maintainability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants