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

Support for a config file #48

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Support for a config file #48

wants to merge 3 commits into from

Conversation

StefMa
Copy link
Collaborator

@StefMa StefMa commented Nov 6, 2024

fixes #47

@rsj-ioki
Copy link

I don't get it, config files are supposed to be used for app config and not for input data. Did I miss the point?

@StefMa
Copy link
Collaborator Author

StefMa commented Nov 11, 2024

I probably went a bit too far with that 🙈
I thought it would be nice to have kind of ever input as a possible config. However I guess this violance a bit of that concept.

So what would you recommend? Make it possible to only configure the tokens and that's it? 🤔

@rsj-ioki
Copy link

IMO you don't really to add support for config files. For the simplicity of this app, everything via CLI should be just fine.

@StefMa
Copy link
Collaborator Author

StefMa commented Nov 11, 2024

Well, "the problem" is, that I use multiple AI providers. Each of them needs another token. Right now I always have to open my password manager, search for the provider and copy paste the token into it.
That's pretty annoying.
So I thought I can simplify this by using a config file where I store the tokens so I don't have to do that process each time again and again...
(ignoring the security problem that I would store api tokens in plain text in disk)

Would you recommend another approach to avoid this? 🤔

@rsj-ioki
Copy link

ahh! then I misunderstood your intentions. As per your last comment using a config to storesettings is fine. I think its the config example that confused me. The github section is taking issue from config too which is not nice as it falls under data

@StefMa
Copy link
Collaborator Author

StefMa commented Nov 13, 2024

So..? Only support for tokens in the config file? 🙃

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.

Introduce configuration file
2 participants