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

chore: remove leftover console.logs #1339

Closed
wants to merge 1 commit into from
Closed

Conversation

43081j
Copy link
Collaborator

@43081j 43081j commented Jul 30, 2024

leftover from some debugging in the v4 branch

@paulmillr
Copy link
Owner

FYI the v4 branch is WIP locally here and ongoing a huge refactoring. It's much more complicated then initially thought, but getting there.

@43081j
Copy link
Collaborator Author

43081j commented Jul 30, 2024

is there somewhere easy to reach you so we could catch up on this stuff?

would be happy to help, and it'd be nice to know what you're changing etc. i don't want to end up duplicating work but want to help get it over the line still

im back for most of august so should have time to help if needed

@paulmillr
Copy link
Owner

@paulmillr paulmillr closed this Jul 31, 2024
@43081j
Copy link
Collaborator Author

43081j commented Aug 1, 2024

that's a great write up 🙏

if you need any help along the way, just let me know

super happy you found the time to get back onto this

@paulmillr
Copy link
Owner

@43081j check out the code for v4.ts - any help in refactoring this is needed.

@43081j
Copy link
Collaborator Author

43081j commented Aug 1, 2024

if you can make a draft PR, I can leave comments too. I think that'd be helpful so I could point out any areas I'm looking at, assuming I do some

@paulmillr
Copy link
Owner

#1342

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