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

Lj3954 #12

Open
zen0bit opened this issue Feb 10, 2024 · 2 comments
Open

Lj3954 #12

zen0bit opened this issue Feb 10, 2024 · 2 comments

Comments

@zen0bit
Copy link

zen0bit commented Feb 10, 2024

sorry for opening issue here. But since discussion is scattered around..

@lj3954 you should create PR with quickget refactoring here.

I like that ❤️

@lj3954
Copy link

lj3954 commented Feb 11, 2024

Our code is completely different, it's not remotely compatible. A plugin made for this version of quickget won't work at all on mine. It also appears to me that we have a very different view of what is being accomplished here. @dabrown645 is trying to refactor quickget just for fun, or to clean up the code a little bit (with the side effect of making implementation of new operating systems more complicated). I'm trying to use it to add new features, including the caching feature I've been thinking about for months.

@zen0bit
Copy link
Author

zen0bit commented Feb 11, 2024

I tested a bit and like it 👍

know that's isn't compatible now, but anyway create PR here (even draft) please, so dabrown can see better implementation..

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

No branches or pull requests

2 participants