schema_list
should support sub-folders.
#1069
Replies: 1 comment
-
Interesting idea. IMHO the idea is worth exploring because the suggested directory layout has the potential of supporting more complex configuration and faciliating code sharing. This change isn't as trivial as "sub-directory support in the path-finding of schema files". We need to answer a series of follow-up questions: where to look for a While files may be structured like so, it may not be a good idea to explicitly write the directory in the configuration. Hopefully I made it obvious that it leads to a philosophy change in how to use the YAML source files: configuration packages are drop-in; the configuration from different sources naturally work together without being modified - this require substantial change in librime because the program was not planned to support such complex use-cases. The current configuration tools work as follows: To support "drop-in" configuration, a recipe should apply to the user's config in a non-destructive way. If we put aside the the fact that Rime is making copies of the information from the downloaded YAML repos before using it, I could try to convince you that the current tooling is triving to work in the way you proposed: |
Beta Was this translation helpful? Give feedback.
-
Current behavior
To install new schema, user have to put all schemas in one folder like
and in
default.custom.yaml
Take a look at install instruction of one of the most popular schema:
https://github.com/KyleBing/rime-wubi86-jidian#%E5%9B%9B%E5%AE%89%E8%A3%85
It's way too many steps, and update isn't elegant (do everything again).
Expected change (as extra behavior)
Let users put schemas in separate folders like
and specify like this
Rational
git clone a-repo
(or a copypasta curl command) without the need to manually copy filesREADME.md
within the schema's folder for extra reference.! The need for overengineered tools like /plum/ is eliminated. Just reuse git or curl.
! This simple change also related to #336
Beta Was this translation helpful? Give feedback.
All reactions