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

Code cleanup #100

Merged
merged 1 commit into from
Dec 21, 2023
Merged

Code cleanup #100

merged 1 commit into from
Dec 21, 2023

Conversation

wweitzel
Copy link
Owner

@wweitzel wweitzel commented Dec 21, 2023

  • Move theme related code/types into their own file called lib/theme.ts
  • Use function for functions instead of const to match rest of codebase

@wweitzel wweitzel force-pushed the refactor branch 3 times, most recently from d4bc232 to f295bbf Compare December 21, 2023 06:19
Copy link

cypress bot commented Dec 21, 2023

Passing run #87 ↗︎

0 11 0 0 Flakiness 0

Details:

Code cleanup
Project: top90-frontend Commit: 9895daa980
Status: Passed Duration: 00:26 💡
Started: Dec 21, 2023 6:22 AM Ended: Dec 21, 2023 6:23 AM

Review all test suite changes for PR #100 ↗︎

@wweitzel wweitzel merged commit 82a0532 into master Dec 21, 2023
5 checks passed
@wweitzel wweitzel deleted the refactor branch December 21, 2023 06:23
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