-
Notifications
You must be signed in to change notification settings - Fork 129
docs: bring back vale rules #1930
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
Changes from all commits
d200058
cbbc894
5938e02
7250fdf
e909c6c
cc6c41f
93a8e00
82f3e5b
df90bb6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,24 @@ | ||
extends: substitution | ||
message: "Use '%s' instead of '%s'." | ||
ignorecase: false | ||
ignorecase: true | ||
level: warning | ||
scope: text | ||
swap: | ||
Apify Dashboard: Apify Console | ||
apify freelancers: Apify freelancers | ||
Apify Platform: Apify platform | ||
'(?:[Tt]he\s)?[Aa]pify\sproxy': Apify Proxy | ||
# Product name changes | ||
'apify dashboard': Apify Console | ||
'apify console': Apify Console | ||
'apify store': Apify Store | ||
'the apify console': Apify Console | ||
'the apify store': Apify Store | ||
|
||
# Capitalization fixes | ||
'apify freelancers': Apify freelancers | ||
'apify platform': Apify platform | ||
|
||
# Proxy variations (simplified) | ||
'apify proxy': Apify Proxy | ||
'the apify proxy': Apify Proxy | ||
|
||
# Common abbreviations | ||
circa: approx. | ||
'ca\.': circa |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,9 @@ | ||
extends: existence | ||
message: "The word '%s' should always be capitalized." | ||
extends: substitution | ||
message: "Use '%s' instead of '%s'." | ||
ignorecase: false | ||
level: error | ||
tokens: | ||
- '\bactor\b' | ||
- '\bactors\b' | ||
nonword: true | ||
scope: text,~alt,~code | ||
swap: | ||
'\bactor\b': Actor | ||
'\bactors\b': Actors |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -84,10 +84,12 @@ In the next step, we’ll explore the results in more detail. | |
### Step 3: Explore the Actor | ||
|
||
Let's explore the Actor structure. | ||
|
||
<!-- vale Apify.Capitalization = NO --> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I imagine there are multiple occurrences (or multiple occurrences can happen) of the But it's true that it doesn't solve There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. technically Vale should ignore prose in inline code done with backticks and for the 95% it does... until it doesn't thus this ugly hack There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. and even more so, it should ignore things in headings... but for some god forsaken reason it didn't here |
||
#### The `.actor` folder | ||
|
||
|
||
The `.actor` folder contains the Actor configuration. The `actor.json` file defines the Actor's name, description, and other settings. Find more info in the [actor.json](https://docs.apify.com/platform/actors/development/actor-definition/actor-json) definition. | ||
<!-- vale Apify.Capitalization = YES --> | ||
|
||
#### Actor's `input` | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it make sense to add
NPM
tonpm
also to the swap rules?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I want to try one more package once I deal with those rules and another PR with fixes to almost everything except H1's in Academy called Openly. I think it deals with a lot of those tech terms, might even lead to simplifying
accept.txt