Skip to content

doc rewrite 3#403

Closed
jth-nw wants to merge 2 commits intodevfrom
doc-rewrite-3
Closed

doc rewrite 3#403
jth-nw wants to merge 2 commits intodevfrom
doc-rewrite-3

Conversation

@jth-nw
Copy link
Copy Markdown
Collaborator

@jth-nw jth-nw commented Mar 3, 2026

No description provided.

@jth-nw jth-nw requested a review from a team as a code owner March 3, 2026 00:38
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

docs/passwordsecure/9.2/installation/installation.md

Line 44: Em dash has spaces. Style guide line 409 requires em dashes without spaces. Change "data center — for example" to "data center—for example".

Line 52: Em dash has spaces. Change "Port 1433 TCP — incoming" to "Port 1433 TCP—incoming".

Line 56: Em dash has spaces. Change "Port 443 HTTPS — outgoing" to "Port 443 HTTPS—outgoing".

Line 57: Em dash has spaces. Change "Port 11011 TCP — incoming" to "Port 11011 TCP—incoming".

Line 58: Em dash has spaces. Change "Port 11014 TCP — backup" to "Port 11014 TCP—backup".

Line 59: Em dash has spaces. Change "Port 11016 TCP — incoming" to "Port 11016 TCP—incoming".

Line 60: Em dash has spaces. Change "Port 11018 TCP — incoming" to "Port 11018 TCP—incoming".

Line 61: Em dash has spaces. Change "Port 1433 TCP — outgoing" to "Port 1433 TCP—outgoing".

Line 65: Em dash has spaces. Change "Port 443 HTTPS — incoming" to "Port 443 HTTPS—incoming".

Line 66: Em dash has spaces. Change "Port 11016 — outgoing" to "Port 11016—outgoing".

Line 67: Em dash has spaces. Change "Port 11018 — outgoing" to "Port 11018—outgoing".

Line 71: Em dash has spaces. Change "Port 11011 TCP — outgoing" to "Port 11011 TCP—outgoing".

Line 72: Em dash has spaces. Change "Port 11018 TCP — outgoing" to "Port 11018 TCP—outgoing".

Line 73: Em dash has spaces. Change "Port 52120 TCP — outgoing" to "Port 52120 TCP—outgoing".


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

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

smaller installations or test environments.
:::warning
For production databases, use a fail-safe cluster. Microsoft SQL Server can replicate data to a
different data center — for example, via WAN. Also dedicate a separate Windows server to each role. Separating roles makes it easier to expand and scale the system later. For smaller
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
different data centerfor example, via WAN. Also dedicate a separate Windows server to each role. Separating roles makes it easier to expand and scale the system later. For smaller
different data centerfor example, via WAN. Also dedicate a separate Windows server to each role. Separating roles makes it easier to expand and scale the system later. For smaller

#### MSSQL Server

- Port 1433 TCP for communication with application server (incoming)
- Port 1433 TCP — incoming communication from the application server
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Port 1433 TCPincoming communication from the application server
- Port 1433 TCPincoming communication from the application server

Comment on lines +56 to +61
- Port 443 HTTPS — outgoing connection to the Netwrix Password Secure license server
- Port 11011 TCP — incoming connections from clients or the IIS web server
- Port 11014 TCP backup service (typically doesn't need to be opened)
- Port 11016 TCP — incoming web services (required only when using the Web Application)
- Port 11018 TCP — incoming real-time updates
- Port 1433 TCP — outgoing communication to SQL Server
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Port 443 HTTPSoutgoing connection to the Netwrix Password Secure license server
- Port 11011 TCPincoming connections from clients or the IIS web server
- Port 11014 TCPbackup service (typically doesn't need to be opened)
- Port 11016 TCPincoming web services (required only when using the Web Application)
- Port 11018 TCPincoming real-time updates
- Port 1433 TCPoutgoing communication to SQL Server
- Port 443 HTTPSoutgoing connection to the Netwrix Password Secure license server
- Port 11011 TCPincoming connections from clients or the IIS web server
- Port 11014 TCPbackup service (typically doesn't need to be opened)
- Port 11016 TCPincoming web services (required only when using the Web Application)
- Port 11018 TCPincoming real-time updates
- Port 1433 TCPoutgoing communication to SQL Server

Comment on lines +65 to +67
- Port 443 HTTPS — incoming access from clients
- Port 11016 — outgoing communication to the application server
- Port 11018 — outgoing real-time updates
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Port 443 HTTPSincoming access from clients
- Port 11016outgoing communication to the application server
- Port 11018outgoing real-time updates
- Port 443 HTTPSincoming access from clients
- Port 11016outgoing communication to the application server
- Port 11018outgoing real-time updates

Comment on lines +71 to +73
- Port 11011 TCP — outgoing communication to the application server
- Port 11018 TCP outgoing
- Port 52120 TCP — outgoing communication with the add-on
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Port 11011 TCPoutgoing communication to the application server
- Port 11018 TCPoutgoing
- Port 52120 TCPoutgoing communication with the add-on
- Port 11011 TCPoutgoing communication to the application server
- Port 11018 TCPoutgoing
- Port 52120 TCPoutgoing communication with the add-on

@jth-nw
Copy link
Copy Markdown
Collaborator Author

jth-nw commented Mar 3, 2026

@claude fix all issues

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 3, 2026

Claude finished @jth-nw's task in 1m 16s —— View job


Fixing all issues

  • Read the installation.md file
  • Fix all 14 em dash spacing issues
  • Run Vale to verify fixes
  • Commit and push changes

Summary

Fixed all 14 em dash spacing issues in docs/passwordsecure/9.2/installation/installation.md. All instances of (em dash with spaces) have been replaced with (em dash without spaces) to comply with the Netwrix Style Guide (line 409).

Vale confirms 0 errors, 0 warnings, and 0 suggestions.

Changes have been committed and pushed to the doc-rewrite-3 branch.


| Branch

Remove spaces around em dashes to comply with Netwrix Style Guide.

Co-authored-by: jth-nw <jth-nw@users.noreply.github.com>
@jth-nw jth-nw closed this Mar 3, 2026
@jth-nw jth-nw deleted the doc-rewrite-3 branch March 16, 2026 22:25
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.

1 participant