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

New fork, refactor, does it make sense? #632

Open
etet100 opened this issue Apr 9, 2024 · 6 comments
Open

New fork, refactor, does it make sense? #632

etet100 opened this issue Apr 9, 2024 · 6 comments

Comments

@etet100
Copy link

etet100 commented Apr 9, 2024

Hi. I wanted to add some minor functionality, and after reviewing the code I ended up STARTING to rewrite much of it from scratch. But at some stage I came to the conclusion that it might not make sense.

So a question, mainly to people who have created/create forks of the Candle app themselves. In your opinion, does it make sense to rebuild it completely? The idea is to build it modularly, separate the UI from the logic (to allow easy switching of different UIs) and actually rewrite much of the code.

I wonder if you would see something in such a project that would make you abandon your own forks in favor of such a refactored version. And most importantly - would you be willing to add something from yourselves to such a version?

@barkermn01
Copy link

barkermn01 commented Apr 21, 2024

Actually, at that stage, you're not forking for a merge but rather undertaking a complete rewrite, which means that none of the work others are currently doing can be merged in, thus defeating the purpose of using git.

The optimal approach would be to initiate a new git repository from scratch and transfer over the necessary code, ensuring that your license and README.md files acknowledge this project. Additionally, your new project must also be licensed under GPLv3, as this is a stipulation of the GPL V3 License employed by this software.

@etet100
Copy link
Author

etet100 commented Apr 21, 2024

Actually, I don't think that's quite what I meant. I never planned to merge it to the original (dead) Candle. It was more about the point of creating a new app at all. And I know that the work already done will not necessarily be easy to merge into my fork. But still, someone may feel like doing it.

@mar0x
Copy link

mar0x commented Apr 22, 2024

Hello,
Although it's a matter of wording, I would rather said the project is "suspend" :) We all use it, fork it, patch it, try to contribute with PR. There are not too much users who want and able (at the same time) to maintain Candle.

@etet100
Copy link
Author

etet100 commented Apr 22, 2024

You know... hundreds of open issues, dozens of PRs. For me, it's more than "suspended." And because of this, X forks were created. That's why I was wondering if it's worth rewriting it (which I'm already doing) and making sure the project lives.

@mar0x
Copy link

mar0x commented Apr 22, 2024

In case of Candle issues means people (still) using it and PRs means people tries to improve it. I agree it would be nice to have someone who continue to maintain it, fix issues, accept and review PRs..

All above does not mean you should not rewrite it. I wonder you asking this question in a "dead Candle" issue :)

Consider to review existing forks list here, maybe you find something useful for your project.

@barkermn01
Copy link

My whole thing was a make a copy not a fork, because your repo's compatibility with this repos code base will be shot from what you have made sound like a very good idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants