-
Notifications
You must be signed in to change notification settings - Fork 63
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
Project status? #64
Comments
this project is dead, he is no more active over 1.5 years |
Is there any active fork? The author didn't respond in a long time, so he isn't available. |
Or is there a good alternative? Everything I have tried so far is buggy and crash prone. |
I did not find any alternative use this two patch, with this patchs works wigdet again for me under Qt 5.15: |
I'd be happy to fork it and maintain it. I don't really know JavaScript that well, but I would be happy to at least review and test PRs. |
just do it ;-) |
I've actually forked it here quite some time ago exactly to merge some patches and not do it manually. It's a gitlab instead of github though but I'm happy to merge and review PRs although my experience is also limited. Also happy to grant permissions for others, so it's not a one man show. Let me know if you're interested |
nice @agurenko . i asked for switching to your fork https://bugs.archlinux.org/task/67375 |
@agurenko I've tested your fork on my system here, and I'm still getting that crash when opening the config window. Using the upstream repo with the patches listed above (see these two patches), the crash does not occur. I can dig into this more when I have some free time, but otherwise, would you mind investigating and merging that patch in with your fork? I'd be happy to switch the AUR package to use it. |
@agurenko Okay, I've managed to merge in those changes to your fork. I'll be creating a PR for you shortly. Once that's merged, I'll switch the AUR package over. I'll also create a BIG BOLD LETTERS issue in this repo to make it easier for people to find your fork when browsing this repo. EDIT: https://gitlab.com/agurenko/plasma-applet-thermal-monitor/-/merge_requests/4 |
There are a bunch of issues and pull requests with fixes that are sitting around for a while. @kotelnik: are you still maintaining the project?
The text was updated successfully, but these errors were encountered: