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

Update Last Man Standing v1.7.2 #7206

Merged
merged 7 commits into from
Jan 7, 2025
Merged

Update Last Man Standing v1.7.2 #7206

merged 7 commits into from
Jan 7, 2025

Conversation

pwatts6060
Copy link
Contributor

  • Minor changes to autodetect new variant of terminator bots

@runelite-github-app
Copy link

runelite-github-app bot commented Jan 4, 2025

@LlemonDuck
Copy link
Contributor

Not new to this PR but can you make sure this gets removed on shutDown https://github.com/pwatts6060/runelite-plugins/blob/lms/src/main/java/com/lmsnotifier/LMSPlugin.java#L181

@LlemonDuck LlemonDuck added the waiting for author waiting for the pr author to make changes or respond to questions label Jan 7, 2025
@pwatts6060
Copy link
Contributor Author

pwatts6060 commented Jan 7, 2025

Not new to this PR but can you make sure this gets removed on shutDown https://github.com/pwatts6060/runelite-plugins/blob/lms/src/main/java/com/lmsnotifier/LMSPlugin.java#L181

It is a bit confusing but it is removed already in a subroutine called in shutdown. ItemUpgrade.reset(). I'll refactor it

@runelite-github-app runelite-github-app bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Jan 7, 2025
@LlemonDuck
Copy link
Contributor

Thanks! That's easier to follow

@LlemonDuck LlemonDuck merged commit 5476c3f into runelite:master Jan 7, 2025
2 of 3 checks passed
@pwatts6060 pwatts6060 deleted the lms branch January 7, 2025 01:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants