-
Notifications
You must be signed in to change notification settings - Fork 0
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
Conversation
This is looking good to me. |
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. |
watch is unnecessary as it's fairly simple to add -- --watch to the end. it's almost the same amount of typing
f94fd5d
to
3f2e95e
Compare
Visit the preview URL for this PR (updated for commit 08ba205): |
ca9dc1d
to
a0a1228
Compare
a0a1228
to
e715682
Compare
🤯
i can go either way on having more run scripts or passing the watch flag to an existing one 🤷🏼