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

Don't store app state in configuration file #238

Open
matthiasbeyer opened this issue Oct 20, 2021 · 1 comment
Open

Don't store app state in configuration file #238

matthiasbeyer opened this issue Oct 20, 2021 · 1 comment

Comments

@matthiasbeyer
Copy link

Hi,

I'm tracking my configuration files in git. Using cointop with this approach is not nice (possible, but not nice) because cointop writes application state (favourite coins for example) to the configuration file... where it does not belong (IMHO).

State should be written to a state directory (XDG defines XDG_STATE_HOME, which would be a good place, IMO).

Could you change that? 😄

Otherwise this is a nice application! 👍

@lyricnz
Copy link
Collaborator

lyricnz commented Oct 20, 2021

I kindof agree - it's annoying how cointop writes to the config file, and looses any comments or formatting, or anything it doesn't understand. We would need to define out the config->state override works carefully, though.

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

2 participants