Skip to content

Update docs auditor datasources add feedback link#411

Merged
nwnikacc merged 9 commits intonetwrix:devfrom
nwnikacc:update_docs_auditor_datasrc_feedback
Mar 5, 2026
Merged

Update docs auditor datasources add feedback link#411
nwnikacc merged 9 commits intonetwrix:devfrom
nwnikacc:update_docs_auditor_datasrc_feedback

Conversation

@nwnikacc
Copy link
Copy Markdown
Contributor

@nwnikacc nwnikacc commented Mar 4, 2026

Doc Task 394098: Add "Contact us" to Supported Data Sources

nwnikacc added 3 commits March 4, 2026 12:51
Added details about monitoring operations on Azure Files and prerequisites for configuration.
Added a section for sharing ideas and feedback on data sources.
@nwnikacc nwnikacc requested a review from a team as a code owner March 4, 2026 14:55
Updated section heading and link text for clarity.
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/auditor/10.8/configuration/overview.md

Line 42: Remove colon from heading. Headings should not end with punctuation. Change "## Suggest a New Data Source:" to "## Suggest a New Data Source"

Line 40-41: Excessive blank lines. Use only one blank line before the heading, not two.


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.

Comment thread docs/auditor/10.8/configuration/overview.md Outdated
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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/auditor/10.8/configuration/overview.md

Line 44: Uses first person "we" in "so we can discuss your feedback and assist you further". The style guide requires second person or imperative mood for instructions. Rewrite to "to discuss your feedback and get assistance".


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.

Comment thread docs/auditor/10.8/configuration/overview.md
Comment thread docs/auditor/10.8/configuration/overview.md Outdated
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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/auditor/10.8/configuration/overview.md

Line 44: Uses "we" to refer to Netwrix, violating the style guide rule "Avoid using 'we' to refer to Netwrix." The phrase "so we can discuss your feedback and assist you further" should be rewritten without first person plural. Suggested fix: "to discuss your feedback and get assistance."


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.

Comment thread docs/auditor/10.8/configuration/overview.md
Comment thread docs/auditor/10.8/configuration/overview.md Outdated
Comment thread docs/auditor/10.8/configuration/overview.md Outdated
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/auditor/10.8/configuration/overview.md

Line 41: Heading doesn't follow sentence case. "Suggest a New Data Source" should be "Suggest a new data source" (capitalize only the first word and proper nouns per style guide lines 537-541).

Line 44: The phrase "and get assistance" is redundant. "Discuss your feedback" already implies seeking help. Remove unnecessary words per style guide line 32.

docs/auditor/10.8/configuration/azurefiles/overview.md

None.


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.

Comment thread docs/auditor/10.8/configuration/overview.md
Comment thread docs/auditor/10.8/configuration/overview.md
@nwnikacc nwnikacc merged commit fa40dc2 into netwrix:dev Mar 5, 2026
5 checks passed
@nwnikacc nwnikacc deleted the update_docs_auditor_datasrc_feedback branch March 5, 2026 19:18
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.

3 participants