Skip to content

Training mar09#637

Merged
hilram7 merged 4 commits intodevfrom
training-mar09
Mar 23, 2026
Merged

Training mar09#637
hilram7 merged 4 commits intodevfrom
training-mar09

Conversation

@robin-stefani
Copy link
Collaborator

Added NPS 5801 and updated learning paths and affected courses.
Added new NPC course and learning paths.
Added latest 1Secure and NPS LTTs.

@github-actions
Copy link
Contributor

Documentation PR Review

Editorial Review

  • Accuracydocs/customer/training/learn-about/pingcastle.md, line 15: "It contains the following courses" but only one course is listed. The parallel file product/pingcastle.md correctly uses the singular "course." Suggested fix: "It contains the following course:"

  • Voicedocs/customer/training/learn-about/pingcastle.md, line 15: "you will be introduced to" is passive. Suggested fix: "This learning path introduces you to <NPC />."

  • Wordinesssrc/training/privilege-secure/5801.md, line 9: "provides an understanding of how to implement this product" is indirect. Suggested fix: "explains how to implement this product"

  • Grammarsrc/training/privilege-secure/5801.md, line 10: "It is comprised of the following modules" — "comprised of" is nonstandard; "comprise" means "consist of" and takes no preposition. Suggested fix: "It comprises the following modules:"

Summary

4 editorial suggestions across 2 files. Vale and Dale issues are auto-fixed separately.


What to do next:
Comment @claude on this PR followed by your instructions. For example:

  • @claude fix all issues — fix all editorial issues
  • @claude reorganize the prerequisites section
  • @claude help improve the flow of this document

Automated fixes are only available for branches in this repository, not forks.

@robin-stefani robin-stefani requested a review from hilram7 March 23, 2026 20:54
@hilram7 hilram7 merged commit c5fa9fa into dev Mar 23, 2026
8 of 9 checks passed
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