Skip to content

Conversation

honzajavorek
Copy link
Collaborator

@honzajavorek honzajavorek commented Sep 8, 2025

According to my tests, this fixes Vale checking. Current vale.ini disables checking in most of the files, one of the regexes probably being too greedy. Exctracted from #1890

I couldn't confirm that enabling specific Apify rules like

Apify.H1 = YES
Apify.Apify = YES
Apify.Capitalization = YES
Apify.Languages = YES

is necessary, Vale seems to pick them up correctly on my machine.

According to my tests, this fixes Vale checking.
Current vale.ini disables checking in most
of the files, one of the regexes probably being
too greedy. Exctracted from #1890

I couldn't confirm that enabling specific Apify rules like

```ini
Apify.H1 = YES
Apify.Apify = YES
Apify.Capitalization = YES
Apify.Languages = YES
```

is necessary, Vale seems to pick them up correctly on my machine.
@honzajavorek honzajavorek requested a review from TC-MO September 8, 2025 09:44
@honzajavorek honzajavorek added documentation Improvements or additions to documentation. adhoc Ad-hoc unplanned task added during the sprint. t-docs Issues owned by technical writing team. labels Sep 8, 2025
@apify-service-account
Copy link

Preview for this PR was built for commit 08290a2 and is ready at https://pr-1901.preview.docs.apify.com!

@honzajavorek honzajavorek merged commit 0043ffe into master Sep 8, 2025
10 checks passed
@honzajavorek honzajavorek deleted the honzajavorek/fix-vale-checking branch September 8, 2025 09:50
patrikbraborec pushed a commit that referenced this pull request Sep 10, 2025
According to my tests, this fixes Vale checking. Current `vale.ini`
disables checking in most of the files, one of the regexes probably
being too greedy. Exctracted from
#1890

I couldn't confirm that enabling specific Apify rules like

```ini
Apify.H1 = YES
Apify.Apify = YES
Apify.Capitalization = YES
Apify.Languages = YES
```

is necessary, Vale seems to pick them up correctly on my machine.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adhoc Ad-hoc unplanned task added during the sprint. documentation Improvements or additions to documentation. t-docs Issues owned by technical writing team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants