Skip to content

Comments

Match refactors#48

Merged
spottseng merged 4 commits intomainfrom
initial-match-refactors
Apr 24, 2025
Merged

Match refactors#48
spottseng merged 4 commits intomainfrom
initial-match-refactors

Conversation

@spottseng
Copy link
Owner

Summary

Provide a brief summary of the changes in this PR and link to the relevant issue(s) if applicable.

  • guts all the methods in the MatchAnalytics and MatchPage files
  • makes MatchAnalytics as class and makes the data a class level variable
  • adds test coverage to the helper methods within MatchAnalytics
  • adds new visualizations to the MatchPage

@spottseng spottseng merged commit 3386850 into main Apr 24, 2025
1 check passed
@spottseng spottseng deleted the initial-match-refactors branch April 24, 2025 14:45
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