Skip to content

Refactor before release#7

Merged
wizzomafizzo merged 8 commits intomainfrom
wizzo/release-refactor
Jan 20, 2024
Merged

Refactor before release#7
wizzomafizzo merged 8 commits intomainfrom
wizzo/release-refactor

Conversation

@wizzomafizzo
Copy link
Member

@wizzomafizzo wizzomafizzo commented Jan 19, 2024

WIP of refactoring everything so we're better set up for future multi platform support. Should eventually be easier to maintain as well.

  • Move cmd files into separate modules
  • Remove dependency on mrext logger
  • Further split the daemon/service function
  • Identify and move all mister-specific code to platforms module

@wizzomafizzo
Copy link
Member Author

Not perfect, but I think it's a pretty decent start for cleaning everything up. There still needs to be some work done to set up an interface between the launcher and platform to remove the last of the mister-specific code, but otherwise it's pretty clean now.

@wizzomafizzo wizzomafizzo marked this pull request as ready for review January 20, 2024 10:32
@wizzomafizzo wizzomafizzo merged commit b5943c3 into main Jan 20, 2024
@wizzomafizzo wizzomafizzo deleted the wizzo/release-refactor branch January 20, 2024 10:32
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.

1 participant