Skip to content

Refactor 1#8

Merged
patinthehat merged 93 commits intomainfrom
refactor-1
Aug 31, 2023
Merged

Refactor 1#8
patinthehat merged 93 commits intomainfrom
refactor-1

Conversation

@patinthehat
Copy link
Member

@patinthehat patinthehat commented Aug 18, 2023

This PR primarily decouples the global App variable from several classes and moves them into separate packages. It also implements an initial set of unit tests and major code refactoring/cleanup in several areas, plus several minor bug fixes.

Includes major refactoring, interface implementations, reorganization and refactoring of most features, implementation of enums where appropriate, addition of workflow state to track the currently running task and task execution history, and more.

@codecov
Copy link

codecov bot commented Aug 18, 2023

Codecov Report

❗ No coverage uploaded for pull request base (main@7fdc92d). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #8   +/-   ##
=======================================
  Coverage        ?   26.19%           
=======================================
  Files           ?       18           
  Lines           ?     2012           
  Branches        ?        0           
=======================================
  Hits            ?      527           
  Misses          ?     1444           
  Partials        ?       41           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@patinthehat patinthehat changed the title Refactor (1) Refactor 1 Aug 19, 2023
@patinthehat patinthehat merged commit f21392e into main Aug 31, 2023
@patinthehat patinthehat deleted the refactor-1 branch August 31, 2023 10:58
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