Skip to content

UA Text Review and Updated Docs Style Guide#169

Merged
longieirl merged 1 commit into
mainfrom
ua-text-review
Jun 16, 2026
Merged

UA Text Review and Updated Docs Style Guide#169
longieirl merged 1 commit into
mainfrom
ua-text-review

Conversation

@lfindlaysap

Copy link
Copy Markdown
Member

Pull Request

Updates all the .md and other text files in accordance with the UI text writing guidelines and SAP Style Guide for Technical Communication. Updated the style guide and skill accordingly.

Checklist

  • Sample tested and runs without errors - Didn't touch any sample code. Only documentation.
  • No sensitive data included

@lfindlaysap lfindlaysap requested a review from longieirl June 10, 2026 20:11
@lfindlaysap lfindlaysap self-assigned this Jun 10, 2026
@lfindlaysap lfindlaysap added the documentation Improvements or additions to documentation label Jun 10, 2026
@github-actions

Copy link
Copy Markdown

🔗 Link Check Results

The link checker has completed. This is an optional check and will not block the PR.

View workflow run for details

If broken links were found, please review them at your convenience.


Last updated: Run #243 at 2026-06-09T08:49:36Z

@longieirl

Copy link
Copy Markdown
Member

Thank you very much @lfindlaysap

@longieirl longieirl merged commit 79b1f36 into main Jun 16, 2026
6 checks passed
@github-actions github-actions Bot deleted the ua-text-review branch June 16, 2026 08:40
longieirl added a commit that referenced this pull request Jul 6, 2026
Add new rules from UA Text Review (PR #169) that were missing:

structural.js:
- Prerequisites bullets must start with "You have" or "You are"
- License heading must be H2
- Headings must not end with colon or question mark
- "Step N:" and "Issue N:" heading prefixes are banned
- Discouraged heading names flagged (Get Support -> Help and Support, etc.)

formatting.js:
- Title case now applies to all heading depths (not just H1/H2)
- toTitleCase updated: "with" added to lowercase prepositions; participles
  ("Using", "Running") now correctly capitalized as verbs/adjectives
- shouldUseTitleCase applies to all headings (not a keyword subset)

content.js:
- Semicolons in prose flagged (replace with colon or period)
- Single-item ordered lists flagged (convert to bullet or prose)
- "There are N / the following N" list count in prose flagged

AGENTS.md:
- Documentation standards section updated to reflect all PR #169 rules

Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants