Skip to content

Adjust Pipeline/Step namespaces - #148

Open
Toastbrot236 wants to merge 9 commits into
LittleBigRefresh:mainfrom
Toastbrot236:pipeline-no-nonsense-1
Open

Toastbrot236 wants to merge 9 commits into
LittleBigRefresh:mainfrom
Toastbrot236:pipeline-no-nonsense-1

Conversation

@Toastbrot236

Copy link
Copy Markdown
Contributor

This PR is the first of multiple PRs intending to refactor and change various things on Refresher, in the hopes of improving maintainability and making it easier to add additional features.

This PR does the following:

  • Pipelines and Steps were moved into multiple separate namespaces and directories, since we have many of them now. This way they're not as messy anymore in my opinion.
  • LogType is no longer a global import, so all log calls now have to specify their category enum value as e.g. LogType.PS3 instead of just PS3
  • The Pipeline.Id property was removed since it's not used, and will likely not be used anytime soon
  • Steps now have a Name property which contains a short, user-friendly description on what the step does. In the future, the GUI should show the name of the current step being executed using this property.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant