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.

Sign up for GitHub

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

Jump to bottom

Simple patch for using GetModule<T>(); instead of array selectors. #2188

Closed
wants to merge 3 commits into from
Closed

Simple patch for using GetModule<T>(); instead of array selectors. #2188

wants to merge 3 commits into from

Conversation

iamalexrouse
Copy link
Member

@iamalexrouse iamalexrouse commented Oct 30, 2023 •
edited
Loading

Pull Request #2188 Info/Notes

Changes

  • Switched out the array system used for the stat modules with GetModule<T>();.

More Notes

  • None

End of PR

@iamalexrouse
Copy link
Member Author

Notes

Reviewers

Changes

  • Rebased PR so it's up-to-date and shouldn't cause merge conflicts.

Notes

Everything should be all good for a Dev CI run.

@iamalexrouse iamalexrouse mentioned this pull request Nov 9, 2023
Merged
@Misaka-ZeroTwo
Copy link
Member

Duplicate #2171

@iamalexrouse iamalexrouse deleted the getmodule-d-patch branch November 13, 2023 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@louis1706 louis1706 Awaiting requested review from louis1706

Assignees

@iamalexrouse iamalexrouse

Labels
regarding-api
Projects
EXILED Workflow
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@iamalexrouse @Misaka-ZeroTwo @louis1706 @Thundermaker300