Skip to content

[Code] WebInterfaceList Global to Singleton Cleanup#4935

Merged
Akkadius merged 1 commit intomasterfrom
kinglykrab/webinterfacelist-global-to-singleton
Jun 25, 2025
Merged

[Code] WebInterfaceList Global to Singleton Cleanup#4935
Akkadius merged 1 commit intomasterfrom
kinglykrab/webinterfacelist-global-to-singleton

Conversation

@Kinglykrab
Copy link
Copy Markdown
Contributor

Description

This is part of a larger effort to remove global variables from the codebase. Eventually singletons can be cleaned up to be passed explicitly through dependency injection.

For now this encapsulates dependencies far better.

Type of change

  • Code Cleanup

Testing

TBD

Checklist

  • [] I have tested my changes
  • I have performed a self-review of my code. Ensuring variables, functions and methods are named in a human-readable way, comments are added only where naming of variables, functions and methods can't give enough context.
  • I own the changes of my code and take responsibility for the potential issues that occur

@Akkadius Akkadius force-pushed the kinglykrab/webinterfacelist-global-to-singleton branch from e9b9eb7 to b8f9b31 Compare June 25, 2025 19:49
Copy link
Copy Markdown
Contributor

@Akkadius Akkadius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Manually rebased and built

@Akkadius Akkadius merged commit c99bda3 into master Jun 25, 2025
0 of 2 checks passed
@Akkadius Akkadius deleted the kinglykrab/webinterfacelist-global-to-singleton branch June 25, 2025 19:49
@Akkadius Akkadius mentioned this pull request Jun 25, 2025
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.

3 participants