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

Incorrect lint:zwave error description #7170

Open
3 tasks done
AlCalzone opened this issue Sep 16, 2024 Discussed in #7169 · 0 comments
Open
3 tasks done

Incorrect lint:zwave error description #7170

AlCalzone opened this issue Sep 16, 2024 Discussed in #7169 · 0 comments

Comments

@AlCalzone
Copy link
Member

Discussed in #7169

Originally posted by @MartinHjelmare September 15, 2024

Checklist

  • I have read and followed the above instructions

  • I have checked the troubleshooting section and my problem is not described there.

  • I have read the changelog and my problem was not mentioned there or the fix did not work.

Describe the issue

What is happening?
When running yarn run lint:zwave or alternatively using the same tool inside vscode on the fly, the error description when formatting a command class incorrectly doesn't explain what is wrong correctly. It says "Parameter descriptions must be Sentence case".

What did you expect to happen instead?
It should say that command classes must be Title Case.

Steps to reproduce the behavior:

  1. Add a device config file with options:
...
"options": [
{
  "label": "Central scene mode",
  "value": 2
}
]
  1. Run: yarn run lint:zwave
  2. See error:
error  Parameter descriptions must be in Sentence case  @zwave-js/consistent-config-string-case
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

No branches or pull requests

1 participant