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

Vale does not report the correct replacement term #802

Closed
1 task done
aireilly opened this issue Apr 8, 2024 · 0 comments
Closed
1 task done

Vale does not report the correct replacement term #802

aireilly opened this issue Apr 8, 2024 · 0 comments

Comments

@aireilly
Copy link

aireilly commented Apr 8, 2024

Check for existing issues

  • Completed

Environment

$ vale -v
vale version 3.4.0

Fedora Linux 39 (Workstation Edition)

Describe the bug / provide steps to reproduce it

Vale does not report the correct replacement term when the replacement term is the same as the error term with different capitalization.

Example Vale rule that shows this error:

---
extends: substitution
ignorecase: false
level: error
message: "Use %s rather than %s."
action:
  name: replace
swap:
  Mutual TLS: mutual TLS
  VNIC: vNIC

Example output:

┌───── ~/vale-test (vale-test-4)
$ vale -v
vale version 3.4.0

┌───── ~/vale-test (vale-test-4)
$ vale .

 modules/vale-test.adoc
 3:108  error  Use 'Mutual TLS' rather than    RedHat.Test 
               Mutual TLS.                                 
 5:16   error  Use 'VNIC' rather than VNIC.    RedHat.Test 

✖ 2 errors, 0 warnings and 0 suggestions in 2 files.

Vale CLI and SublimeText + vale-ls both show the error. Clone aireilly/vale-test to replicate:

git clone -b vale-test-4 https://github.com/aireilly/vale-test && cd vale-test && vale .
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants