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

sample config.yaml contains invalid fields #922

Closed
dimitrilw opened this issue Oct 1, 2023 · 6 comments
Closed

sample config.yaml contains invalid fields #922

dimitrilw opened this issue Oct 1, 2023 · 6 comments
Labels
kind/doc Document update

Comments

@dimitrilw
Copy link

  • os: MacOS Sonoma 14.0, current as of time of this post
  • lsd --version: 1.0.0, installed 10 mins ago via Homebrew
  • echo $TERM: xterm-256color
  • echo $LS_COLORS: none -- empty return value (see pic, below)

Expected behavior

Process all fields given in sample config.yaml content.

Actual behavior

lsd: Configuration file /Users/dimitri/.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 143 column 1.

...and:

lsd: Configuration file /Users/dimitri/.config/lsd/config.yaml format error, unknown field `truncate-owner`, expected one of...
image
@zwpaper
Copy link
Member

zwpaper commented Oct 2, 2023

Hi @dimitrilw , the master is for the latest dev features, you can check 1.0 by changing the code to v1.0.0 tag

Can you also help adding a notice under configuration in readme?

@zwpaper
Copy link
Member

zwpaper commented Oct 2, 2023

/kind support

@muniu-bot
Copy link

muniu-bot bot commented Oct 2, 2023

@zwpaper: The label(s) kind/support cannot be applied, because the repository doesn't have them.

In response to this:

/kind support

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.

@dimitrilw
Copy link
Author

Thank you for that explanation. ...it is always a 50/50 chance on "is main (aka master) prod, or is main dev" 😄 ...anyway, since your repo's primary readme is the main branch, that's what users (like me) will probably expect to match the released version -- worth adding header statement pointing peeps to old readme files hidden in release branches

@zwpaper zwpaper added the kind/doc Document update label Oct 7, 2023
@rlovell3
Copy link

The config file issues just tripped me up as well on a fresh (10-minutes ago) install.

@zwpaper
Copy link
Member

zwpaper commented Nov 22, 2023

sorry for that @rlovell3, fixed now by #960

@zwpaper zwpaper closed this as completed Nov 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/doc Document update
Projects
None yet
Development

No branches or pull requests

3 participants