Skip to content
Snippets Groups Projects

Add new FrontmatterTitle Vale rule and fix errors raised by new rule

Merged Evan Read requested to merge eread/add-frontmattertitle-vale-rule-and-fix-errors into main
All threads resolved!

Why is this change being made?

From https://gitlab.com/gitlab-com/content-sites/docsy-gitlab/-/blob/main/content/docs/frontmatter.md, we need every content page to have its level 1 header declared in frontmatter.

This Vale rule ensures that every page follows this frontmatter rule.

Author and Reviewer Checklist

Please verify the check list and ensure to tick them off before the MR is merged.

  • Provided a concise title for this Merge Request (MR)
  • Added a description to this MR explaining the reasons for the proposed change, per say why, not just what
    • Copy/paste the Slack conversation to document it for later, or upload screenshots. Verify that no confidential data is added, and the content is SAFE
  • Assign reviewers for this MR to a code owner, typically a handbook maintainer
  • For transparency, share this MR with the audience that will be impacted
    • #handbook Slack channel if there is any user impact
    • #mr-buddies Slack channel if changes may impact more users
    • #whats-happening-at-gitlab linking to this MR if a large number of team members are impacted

Commits

%{all_commits}


Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Evan Read
  • Evan Read requested review from @cynthia

    requested review from @cynthia

  • Cynthia "Arty" Ng approved this merge request

    approved this merge request

  • Evan Read changed milestone to %17.7

    changed milestone to %17.7

  • Cynthia "Arty" Ng enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • Cynthia "Arty" Ng resolved all threads

    resolved all threads

  • mentioned in commit e40174f4

  • Please register or sign in to reply
    Loading