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

Move default history location to ~~state/ #36

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nbraud
Copy link

@nbraud nbraud commented Apr 29, 2024

  • Use state directory as the default history_path, see Store history in application state directory #35
  • Automatically migrate history from ~~/ to ~~state/
    Setting history_path to the old value will prevent automatic migration
    Tested on Linux (NixOS)

@nbraud
Copy link
Author

nbraud commented Apr 29, 2024

I considered what happened if ~~/ and ~~state/ are the same:
in that case, file_info(old) and file_info(new) are either both non-nil or both nil,
so the migration path is not taken either way.

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.

1 participant