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

False positive missing required prop due to the hyphanate function #12011

Closed
kazvaggos opened this issue Sep 23, 2024 · 0 comments · Fixed by #12034
Closed

False positive missing required prop due to the hyphanate function #12011

kazvaggos opened this issue Sep 23, 2024 · 0 comments · Fixed by #12034

Comments

@kazvaggos
Copy link

Vue version

3.4.15

Link to minimal reproduction

https://play.vuejs.org/#eNqNUsFSwjAQ/ZU1F3SG0oOcsDJjHQ56UEad8ZJLaRcaaJOYpIDT6b+bpFBwFIZLJtn3Nvt299XkQcrBukIyIpFOFZMGNJpKjilnpRTKwKMoJcyVKKE3CN3D0Xt3lFMehW2KJduHwVIWiUH7AohSlzbKEx3MhsEskErIe0qMqpASCMcQXQUBfAq10rBhJheVgQQ2ieKML8DkiQGXAkyDwq+KKcwgCP58HVzy92UfR+FRA6RPjE4Fn7PFYKkFt+OpXW1KXHFWoHqVhgmuKRmBRxyWFIXYPPuYE9Pfx9Mc09U/8aXeuhglU4Ua1drK7zCTqAWaFp68v+DW3juwFFlVWPYZ8A21KCqnsaXFFc+s7COeV/vkl2xH86EnW4Nc75vy07TMxvMpsUt3yz/V+kHu7WDo8yhv7BT3hjntrxoynDOOU7sVDc3Oas5jzlVwjF7vytnVx8PYhToFAOZb4ghiIQpM+K5J6HZ8aMhLs8fNef9mbO0vAHV9qNf43Chs0d+WaX4A878YBQ==

Steps to reproduce

You can open the console to see the false positive warning on the above example

What is expected?

It is expected that the hyphanated prop of hasB4BProp would result to has-b-4-b-prop

What is actually happening?

The hyphanated prop of hasB4BProp results to has-b4-b-prop

System Info

System:
    OS: macOS 14.7
  Binaries:
    Node: 20.2.0
    npm: 9.6.6
  Browsers:
    Chrome: 128.0.6613.138
  npmPackages:
    vue: 3.4.15 => 3.4.15

Any additional comments?

Just noticed the false positive warning while the functionality works as expected. Also tested the hyphenate function and found out the reason

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants