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

Added note to not ask for GUI changes #183

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jakjakob
Copy link

Maybe it prevents all the (useless) issues asking for a new/bloated GUI.
(added a line in the future request template saying that the tool is meant to be be minimalistic and to not ask useless/big GUI features/changes)
Should I also add something into the readme?
Maybe a good project for the future is making a Q&A page, that also says that there aren't bi GUI changes in mind. If you think this is a good idea I could make it.

(btw love this tool, keep the good work up)

added a line saying that the tool is meant to be be minimalistic and to not ask useless/big GUI features/changes
@Tokarak
Copy link

Tokarak commented Dec 10, 2023

Funny that a "minimalist" app needs 85mb...

@actuallymentor
Copy link
Owner

Hey @jakjakob!

My apologies for responding so late. The tragedy of open source side projects is that I only check in periodically.

Your PR is very much appreciated. I have mixed feelings about merging it. One the one hand I am 100% on board with the sentiment. On the other hand, I want to leave the space for people to build on the project if they have the time for it.

A recent case study is the battery calibrate functionality. To my mind that doesn't need to be a feature, and those who really care should just script it. On the other hand, if people care enough to make the functionality themselves in a nonobtrusive way (like @KyrosWeb did in #169) then I have no problem merging it.

How would you feel about rephrasing it a bit to include something like "if you really care about a feature, be prepared to built it yourself, and check in beforehand whether it would be merged" or so?

Also, @Tokarak, you are completely unhelpful and needlessly snarky. The CLI is 17KB, and nobody who uses a GUI cares about 85mb of drive usage. If you want a smaller GUI, I genuinely hope you build it.

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.

3 participants