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

refactor(sherlock): convert to typescript #368

Merged
merged 16 commits into from
Nov 13, 2024
Merged

Conversation

steveoh
Copy link
Member

@steveoh steveoh commented Nov 13, 2024

🤯

i can go either way on having more run scripts or passing the watch flag to an existing one 🤷🏼

@steveoh steveoh requested a review from stdavis November 13, 2024 02:47
@steveoh steveoh changed the title refactor(sherlock): conver to typescript refactor(sherlock): convert to typescript Nov 13, 2024
@stdavis
Copy link
Member

stdavis commented Nov 13, 2024

This is looking good to me.

@steveoh
Copy link
Member Author

steveoh commented Nov 13, 2024

It might make sense to do the utilities in this pr since a lot is shared.

@stdavis
Copy link
Member

stdavis commented Nov 13, 2024

It might make sense to do the utilities in this pr since a lot is shared.

Sorry I missed this. Hopefully the utilities changes will be able to be rebased into this PR without any issues.

Copy link
Contributor

github-actions bot commented Nov 13, 2024

Visit the preview URL for this PR (updated for commit 08ba205):
Firebase Preview: https://ut-dts-agrc-kitchen-sink-dev--refactor-sherlock-ts-qmr286g1.web.app
(expires "2024-11-27T23:24:22.397865721Z" )

@steveoh steveoh force-pushed the refactor/sherlock-ts branch from ca9dc1d to a0a1228 Compare November 13, 2024 23:11
@steveoh steveoh marked this pull request as ready for review November 13, 2024 23:12
@steveoh steveoh enabled auto-merge (rebase) November 13, 2024 23:12
@steveoh steveoh merged commit a32ad0f into main Nov 13, 2024
6 checks passed
@steveoh steveoh deleted the refactor/sherlock-ts branch November 13, 2024 23:29
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.

2 participants