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

No more in-page table of contents in blog posts #2133

Merged
merged 13 commits into from
May 14, 2024
13 changes: 13 additions & 0 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,19 @@ jobs:
filter_mode: file
version: 3.4.2

- name: Vale Conf news Linter
uses: errata-ai/vale-action@reviewdog
env:
# Required, set by GitHub actions automatically:
# https://docs.github.com/en/actions/security-guides/automatic-token-authentication#about-the-github_token-secret
GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
with:
files: all
plaindocs marked this conversation as resolved.
Show resolved Hide resolved
vale_flags: "--config=vale/news.ini --glob='docs/conf/**/news/*'"
plaindocs marked this conversation as resolved.
Show resolved Hide resolved
reporter: github-pr-check
fail_on_error: true
filter_mode: file
plaindocs marked this conversation as resolved.
Show resolved Hide resolved
version: 3.4.2

yaml-validate:
runs-on: ubuntu-latest
Expand Down
7 changes: 7 additions & 0 deletions vale/WTD/contents.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
message: "Don't use in-page TOC in newsletter posts"
extends: existence
ignorecase: true
level: error
scope: raw
raw:
- 'contents::'
2 changes: 2 additions & 0 deletions vale/guide.ini
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,6 @@ WTD.inclusivity-avoid = YES
WTD.inclusivity = YES
WTD.branding = YES

WTD.contents = NO

proselint.GenderBias = YES
6 changes: 6 additions & 0 deletions vale/news.ini
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
StylesPath = .
MinAlertLevel = warning

[*.rst]

WTD.contents = YES
1 change: 1 addition & 0 deletions vale/vale.ini
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ MinAlertLevel = warning
BasedOnStyles = WTD, proselint

WTD.headings = NO
WTD.contents = NO

# Initially, some proselint styles were included in vale.
# In later vale versions, those were removed, so now we include
Expand Down
Loading