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

Don't assume we have a mosquito when searching for a default familiar #1366

Merged
merged 2 commits into from
Sep 22, 2023

Conversation

Alium58
Copy link
Member

@Alium58 Alium58 commented Sep 20, 2023

Description

Reported in discord that LoL doesn't have a mosquito. This fixes that by instead of hardcoding a familiar we expect to have, instead search for an actually available familiar

How Has This Been Tested?

Did a normal standard run without issue

Checklist:

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have based my pull request against the main branch or have a good reason not to.

@Alium58 Alium58 changed the title initial commit Don't assume we have a mosquito when searching for a default familiar Sep 20, 2023
@Alium58 Alium58 self-assigned this Sep 20, 2023
Malibu-Stacey
Malibu-Stacey previously approved these changes Sep 21, 2023
@Alium58 Alium58 marked this pull request as ready for review September 22, 2023 19:30
@Alium58 Alium58 merged commit 3bf884c into main Sep 22, 2023
2 checks passed
@Alium58 Alium58 deleted the familiarFinder branch September 22, 2023 22:06
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