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

Rework ULX cleanup commands #17

Open
adamnejm opened this issue Dec 15, 2023 · 1 comment
Open

Rework ULX cleanup commands #17

adamnejm opened this issue Dec 15, 2023 · 1 comment

Comments

@adamnejm
Copy link
Contributor

Rework:

  • Move them into a separate "Cleanup" category, putting everything into TAS Utilities is stupid.
  • Export a function from every file, then make ulx tidy that will just call all of them
@adamnejm adamnejm changed the title Rework ulx cleanup commands Rework ULX cleanup commands Dec 15, 2023
@Derpius
Copy link
Member

Derpius commented Dec 15, 2023

If we have so many that a single category doesn't make sense, and if we want to add more complex logic like composing them into a tidy command, then they'd probably be better extracted into a separate addon

⬆️ This is what we've done with bits of utils as they've grown in size, like buildmode, brainlet and the server monitor

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

No branches or pull requests

2 participants