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

feat(deployer): allow disabling backup_config_files #991

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

Conversation

ksqsf
Copy link
Member

@ksqsf ksqsf commented Mar 15, 2025

虽然可以理解 backup_config_files 最初的目的,但目前的生态下,该功能已经没有什么作用了。

  1. 它并不备份 lua 和 opencc 配置,所以备份得到的配置可能是不能用的
  2. 它会直接覆盖掉之前的配置,不能避免误操作,所以还是无法给人放心的感觉
  3. 拖慢「同步」速度

大部分讲究的用户会用 Plum/Git 等管理配置,再单独「同步」用户词库。这样 backup_config_files 就完全没有用了。而不讲究的用户可能本来就不知道这里有备份。

本 PR 在 installation.yaml 中增加一笔 backup_config_files 选项,置为 false 后则不再执行备份操作。默认值也直接改为 false。至于应否设为 true,还待商榷。

@ksqsf ksqsf requested a review from a team March 15, 2025 15:29
@lotem
Copy link
Member

lotem commented Mar 15, 2025

@fxliang 小狼毫有圖形界面,可能需要修改說明文字,可以簡要說明會同步什麼內容。


@ksqsf
說說 .txt 文件。雖然本來也不會自動同步,但要說明一下,文本數據庫也可以設置成自動保存用戶所做修改,表現得像用戶詞典一樣。

@ksqsf
Copy link
Member Author

ksqsf commented Mar 15, 2025

那就让 .txt 继续备份?或者是不是应该直接把 stabledb 的备份也挪到 user dict sync 过程中?

@lotem
Copy link
Member

lotem commented Mar 16, 2025

那就让 .txt 继续备份?或者是不是应该直接把 stabledb 的备份也挪到 user dict sync 过程中?

我就說說。不容易做吧。
關鍵是怎樣讓用戶知道備份停用了。

@fxliang
Copy link
Contributor

fxliang commented Mar 17, 2025

@fxliang 小狼毫有圖形界面,可能需要修改說明文字,可以簡要說明會同步什麼內容。

似乎未见直接调用这个API吧,这个API是什么作用的?

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