Skip to content

Conversation

@KANAjetzt
Copy link
Member

  • reworked validate_dependencies_and_incompatibilities_conflicts() to work with any arrays
  • renamed it to validate_distinct_mod_ids_in_arrays().
  • Fixed the accidentally removed function call of validate_dependencies_and_incompatibilities_conflicts() in _init()
  • Added validation to check for distinct mod_ids in incompatibilities and optional_dependencies.

closes #192

reworked `validate_dependencies_and_incompatibilities_conflicts()` to work with any arrays, renamed it to `validate_distinct_mod_ids_in_arrays()`. Fixed the accidentally removed function call of `validate_dependencies_and_incompatibilities_conflicts()` in `_init()`. Added validation to check for distinct mod_ids in `incompatibilities` and `optional_dependencies`.

closes GodotModding#192
@KANAjetzt KANAjetzt added bug Something isn't working validation Feature to make things safe and predictable refactor / cleanup Improves readability or maintainability labels Mar 28, 2023
@KANAjetzt KANAjetzt added this to the v6.0.0 milestone Mar 28, 2023
@KANAjetzt KANAjetzt requested review from a team and Qubus0 March 28, 2023 13:32
@KANAjetzt KANAjetzt added bug Something isn't working and removed bug Something isn't working labels Mar 28, 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.

👍

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.

nice

@KANAjetzt KANAjetzt added this pull request to the merge queue Mar 29, 2023
Merged via the queue into GodotModding:development with commit 7f9afed Mar 29, 2023
@KANAjetzt KANAjetzt deleted the refactor_validate_distinct_mod_ids_in_arrays branch March 30, 2023 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working refactor / cleanup Improves readability or maintainability validation Feature to make things safe and predictable

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants