Aidan Reilly
Aidan Reilly
This is now ready for review - the new Vale back reference feature is very useful.
In retrospect, I am going to close this un-merged. Terms like the below perfectly valid and will get caught in this rule. ``` ip-10-0-0-1.us-east-2.compute.internal 775898c3-ecc2-41a4-b98b-a4cd5ae56fd0 2023-09-22 ```
@rolfedh done. Welcome @[email protected] :) On a side note, would be good to review existing rule terms and see if it makes sense to pull out other localization terms under...
Excellent :)
re: "should". IMO would be great to figure out where this is allowed and tune the rule to allow for correct usages.
Considering the above, it might make sense to create a distinct and separate Red Hat Localization rule set. E.g., ``` Styles/ AsciiDoc/ OpenShiftAsciiDoc/ RedHat/ Localization/ ```
```yml Smart State|smart state|Smart state|Smartstate Analysis: SmartState analysis ``` is the fix I think.
Think we need a sequence rule for this one: https://vale.sh/docs/topics/styles/#sequence
partially addressed in https://github.com/redhat-documentation/vale-at-red-hat/pull/409 Needs moving to the CaseSenstiveTerms.yml IMO. `AI|assisted installer: Assisted Installer`
Yes, however, I think we need to change it to: `assisted installer: Assisted Installer` in CaseSensitiveTerms.yml and remove it from TermsErrors.yml AI is commonly used to refer to Artificial Intelligence,...