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

Remove unused imports/locals in economy.lua #6685

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

4z0t
Copy link
Member

@4z0t 4z0t commented Mar 2, 2025

Description of the proposed changes

Remove unused imports/locals in economy.lua.

Checklist

  • Changes are annotated, including comments where useful
  • Changes are documented in the changelog for the next game version

@4z0t 4z0t added the area: ui Anything to do with the User Interface of the Game label Mar 2, 2025
@Hdt80bro
Copy link
Contributor

Hdt80bro commented Mar 3, 2025

Since this can break mod compatibility, we typically move unused imports to the bottom of the file.

@Hdt80bro
Copy link
Contributor

Hdt80bro commented Mar 4, 2025

Oh, and it'd probably be good to leave a comment down there about why those imports are there like here (I think you're still missing one local by the way).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ui Anything to do with the User Interface of the Game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants