lsd-rs / lsd

The next gen ls command

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[Bug]: Example config.yaml is not valid

dcechano opened this issue · comments

Version

  • latest

version

lsd 0.23.1

What OS are you seeing the problem on?

Linux

installation

sudo dnf install lsd

term

No response

ls-colors

No response

What happened?

copy and pasted the example config file from the `READ_ME.md` into a config.yaml and ran `lsd --config-file ~/.config/lsd/config.yaml` and got:

`lsd: Configuration file /home/dylan/.config/lsd/config.yaml format error, unknown field `literal`, expected one of `classic`, `blocks`, `color`, `date`, `dereference`, `display`, `icons`, `ignore-globs`, `indicators`, `layout`, `recursion`, `size`, `permission`, `sorting`, `no-symlink`, `total-size`, `symlink-arrow`, `hyperlink`, `header` at line 139 column 1.

thread 'main' panicked at 'Provided file path is invalid', src/main.rs:126:33
stack backtrace:
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
`

Clearly the parse fails when it stumbles on `literal: false` but I am not sure why. Is `literal` no longer supported? If so I can submit a PR with it removed.

What expected?

I expected the file to parse.

What else?

No response

I caught this too, it looks like the main branch is just ahead of the latest release (release still pinned at 1.0.0, quite a bit new since then.) Probably best to build from source.

the master branch is meant to be the dev branch, please select the tag of your version to view the versioned docs

We have another issue tracking this to add a notice

/close

@zwpaper: Closing this issue.

In response to this:

the master branch is meant to be the dev branch, please select the tag of your version to view the versioned docs

We have another issue tracking this to add a notice

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.