-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Add Vale vocabulary list #1674
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
Add Vale vocabulary list #1674
Changes from all commits
58422b9
1395728
a7284a5
bc90d8d
b0d83b5
ebde684
42cbadd
2dbf7dd
0da112e
ad21854
cd89b1f
3dc38a3
121dc9e
bf70bdb
41c363a
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,5 +1,6 @@ | ||
| StylesPath = .vale/styles | ||
| MinAlertLevel = suggestion | ||
| Vocab = ngrok | ||
| Packages = Google, MDX | ||
| [*.mdx] | ||
| BasedOnStyles = Google | ||
| BasedOnStyles = Google |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,116 @@ | ||
| (API|api)(s)? | ||
| (SDK|sdk)(s)? | ||
| [Dd]ogfood(ing)? | ||
| [Dd]oglab(bing)? | ||
| [Hh]omefooding | ||
| [Hh]omelab(s)? | ||
| [Hh]ostname(s)? | ||
| [Ii]ngress(ed|ing)? | ||
| [Oo]llama | ||
| [Pp]roxying | ||
| [Rr]insed | ||
| [Ss]imple[Ll]ogin | ||
| [Ss]mithery | ||
| [Vv]ijil | ||
| agentic | ||
| anonymized | ||
| authtoken | ||
| automations | ||
| Bazel | ||
| Buildkite | ||
| CloudFront | ||
| config | ||
| configs | ||
| coreruleset | ||
| CR[Ss] | ||
| CRD(s)? | ||
| datacenter(s)? | ||
| Datadog | ||
| DeepSeek | ||
| dev | ||
| devs | ||
| Docusaurus | ||
| doxx(ing)? | ||
| exfiltrate | ||
| footguns | ||
| freshrss | ||
| [Gg]eo | ||
| gists | ||
| GPU(s)? | ||
| Groq | ||
| HAProxy | ||
| hardcoded | ||
| immich | ||
| internet's | ||
| (IP|ip)(s)? | ||
| LLM(s)? | ||
| Lua | ||
| masse | ||
| MCP(s)? | ||
| middleware | ||
| monorepo | ||
| multicloud | ||
| multicluster | ||
| MySQL | ||
| nd | ||
| (nginx|NGINX) | ||
| ngrok | ||
| ngrokker(s)? | ||
| (OAuth|oauth) | ||
| operationalize | ||
| Postgre(s|SQL) | ||
| protonmail | ||
| [Pp]rox(y|ied) | ||
| proxmox | ||
| reframed | ||
| [Rr]epo | ||
| rollout | ||
| shortener | ||
| smidge | ||
| subcommand | ||
| toolset | ||
| Twilio | ||
| umami | ||
| uncopyable | ||
| upstreams | ||
| Vercel | ||
| Vin | ||
| VM(s)? | ||
| (VPN|vpn)(s)? | ||
| WIP(s)? | ||
| Wireguard | ||
| workshopping | ||
| shortener | ||
| [Ss]erverless | ||
| onboarded | ||
| Webflow | ||
| allowlist(s)? | ||
| pluggability | ||
| [Ff]ailover(s)? | ||
| [Dd]ataplane(s)? | ||
| [Cc]ontrolplane(s)? | ||
| [Ss]hinobi | ||
| [Pp]entest(s)? | ||
| [Cc]leartext | ||
| RPC(s)? | ||
| GPTBot | ||
| papercut(s)? | ||
| [Cc]omposable | ||
| architected | ||
| [Rr]untime(s)? | ||
| unregister | ||
| WAF(s)? | ||
| Uppal | ||
| [Pp]laintext | ||
| ASN(s)? | ||
| blocklist | ||
| namespace | ||
| GitHhub | ||
| livestream | ||
| [Kk]ubernetes | ||
| kube | ||
| kubectl | ||
| CIDR(s)? | ||
| discoverability | ||
| minikube | ||
| healthmed |
|
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. This file is generated so the main repo is going to put this back. |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -120,7 +120,7 @@ const ngrok = require('@ngrok/ngrok'); | |
| traffic_policy: '{"on_http_request": [{"actions": [{"type": "oauth","config": {"provider": "google"}}]}]}' | ||
| }); | ||
|
|
||
| // Output ngrok url to console | ||
| // Output ngrok URL to console | ||
|
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. As a passing question, is there any way to make this ignore code snippets? In this case, it's a good change, but I think it'd be weird if you got a vale error for a variable name 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 do need to look into how to fine-tune it that way - I had to remove a rule enforcing lowercase "ngrok" because sometimes it appears in the code as |
||
| console.log(`Ingress established at ${listener.url()}`); | ||
| })(); | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -183,7 +183,7 @@ wildcard domain for that factory's telemetry API. | |
| ## Create cloud endpoints for always-on API and database access | ||
|
|
||
| Since the telemetry API and database must be always accessible, create permanent cloud endpoints that route traffic to their | ||
| internal endpoints. Create an HTTPs cloud endpoint for the API using the ngrok platform API. | ||
| internal endpoints. Create an HTTPS cloud endpoint for the API using the ngrok platform API. | ||
|
|
||
| ```curl | ||
| curl -X POST https://api.ngrok.com/endpoints \ | ||
|
|
@@ -234,7 +234,7 @@ curl -X POST \ | |
| ``` | ||
|
|
||
| <Note> | ||
| Ensure your chosen domain has the proper TLS/SSL certs provisioned in order to create a secure, working connection. | ||
| Ensure your chosen domain has the proper TLS/SSL certs provisioned to create a secure, working connection. | ||
|
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. Nice. "in order to" is the bane of my existence 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. once we turn Vale back on in the dashboard "in order to" should fail the CI check and force you to delete it before merging 😈 |
||
| </Note> | ||
|
|
||
| Now, `app.factory.com` is live—only for this session. | ||
|
|
||
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.
This file is generated so the main repo is going to put this back.