Skip to content

docs: use neutral vocabulary examples - #37

Merged
samonjoat merged 1 commit into
mainfrom
cleanup/public-profile-safety
Aug 3, 2026
Merged

docs: use neutral vocabulary examples#37
samonjoat merged 1 commit into
mainfrom
cleanup/public-profile-safety

Conversation

@samonjoat

Copy link
Copy Markdown
Owner

Replace specific person-name examples in XML documentation with a neutral placeholder. Runtime behavior is unchanged.

Copilot AI review requested due to automatic review settings August 3, 2026 21:01

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates XML documentation examples to use a neutral placeholder name, keeping the public API and runtime behavior unchanged while making docs more generic.

Changes:

  • Replaced a specific person-name example with "Contoso" in AddVocabularyWordAsync parameter documentation.
  • Updated DictionaryEntry.Word and DictionaryEntryType.VocabularyWord XML documentation example lists to remove person-name examples and use "Contoso" instead.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
AirType/Services/Dictionary/IDictionaryManager.cs Updates the word parameter XML doc examples to use a neutral placeholder.
AirType/Models/Dictionary/DictionaryEntry.cs Updates XML doc examples for vocabulary word entries/types to use a neutral placeholder.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: samonjoat <52766763+samonjoat@users.noreply.github.com>
@samonjoat
samonjoat force-pushed the cleanup/public-profile-safety branch from cd1c6fe to fca5c02 Compare August 3, 2026 21:03
@samonjoat
samonjoat merged commit 2ce868f into main Aug 3, 2026
5 checks passed
@samonjoat
samonjoat deleted the cleanup/public-profile-safety branch August 3, 2026 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants