-
Notifications
You must be signed in to change notification settings - Fork 2
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
docs: add configuration examples to readme #42
docs: add configuration examples to readme #42
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may want to also show that you can access files in the root by starting your paths with /
.
I don't think this is a good idea to include right now. This functionality is currently untested and also goes against the intended functionality. |
README.md
Outdated
|
||
Variant expressions, denoted with brackets (`[]`), are a shorthand way to set multiple paths. | ||
|
||
Symlink: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be changed to "This code can be shortened by using a variant expression..." or similar.
|
||
Match expressions, denoted with braces (`{}`), allow to conditionally sync files based on a system's operating system and host. | ||
|
||
Conditionally symlink by os and host: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Explain what this does by giving specific examples using OS & host.
Linting: | ||
|
||
$ cargo check | ||
$ cargo clippy --all-targets --all-features |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may want to add that our automatic CI checks fmt
, clippy
, and test
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may want to also add somewhere that you can use backslashes to escape special characters, e.g. .config/badly\ named\[dir\/\}/config
.
README.md
Outdated
$ git clone [email protected]:plamorg/ambit.git | ||
$ cd ambit | ||
$ cargo install --path . | ||
Symlink `${HOME}/host.txt -> ${AMBIT_REPO_PATH}/a/repo.txt`: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ->
should not be part of the monospaced font.
Symlink `${HOME}/host.txt -> ${AMBIT_REPO_PATH}/a/repo.txt`: | |
Symlink `${HOME}/host.txt` to `${AMBIT_REPO_PATH}/a/repo.txt`: |
README.md
Outdated
* `${HOME}/.config/bat/bat.conf -> ${AMBIT_REPO_PATH}/.config/bat/bat.conf` | ||
* `${HOME}/.config/nvim/init.vim -> ${AMBIT_REPO_PATH}/.config/nvim/init.vim` | ||
* `${HOME}/.config/kitty/kitty.conf -> ${AMBIT_REPO_PATH}/.config/kitty/kitty.conf` | ||
* `${HOME}/.config/kitty/theme.conf -> ${AMBIT_REPO_PATH}/.config/kitty/theme.conf` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same with the ->
s here.
README.md
Outdated
|
||
.config/ambit/config.ambit; | ||
|
||
#### Variant Expression |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#### Variant Expression | |
#### Variant expressions |
This PR primarily adds configuration examples to
README.md
and some general documentation improvements.