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

Config: note "config flush false" when option is not writeable via IRC #1458

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Mikaela
Copy link
Contributor

@Mikaela Mikaela commented May 27, 2021

No description provided.

@progval
Copy link
Owner

progval commented May 27, 2021

Why? That shouldn't be needed unless it takes more than an hour to edit the config and reload it

@Mikaela
Copy link
Contributor Author

Mikaela commented May 27, 2021

I just noticed it was missing and thought that it would be a good thing to document so there is no config getting written and possibly causing conflict/confusion with text editor simultaneously.

Isn't more than an hour a bit long time between flushes if something changes?

@progval
Copy link
Owner

progval commented May 27, 2021

It worked so far, and it gives time to edit the config file like this.

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.

2 participants