Skip to content

typos#282

Open
Linegod wants to merge 1 commit intoCacti:developfrom
Linegod:fix-typos-config-readme
Open

typos#282
Linegod wants to merge 1 commit intoCacti:developfrom
Linegod:fix-typos-config-readme

Conversation

@Linegod
Copy link
Member

@Linegod Linegod commented Mar 11, 2026

Typos in README.md and config.php.dist

Copilot AI review requested due to automatic review settings March 11, 2026 04:48
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR corrects a few typos in end-user documentation/config templates for the Syslog plugin, improving readability and reducing confusion during setup.

Changes:

  • Fixes spelling mistakes in config.php.dist comments.
  • Fixes a spelling mistake in the README.md Grouping feature note.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
config.php.dist Corrects typos in inline comments related to silent install options and incoming table fields.
README.md Fixes a typo in the Grouping feature documentation text.

You can also share your feedback on Copilot code review. Take the survey.

}

/* Uncomment lines below to enable options for silent installation during intall plugin */
/* Uncomment lines below to enable options for silent installation during install plugin */
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

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

The comment is still grammatically incorrect: “during install plugin” reads awkwardly. Consider changing it to “during plugin installation” (or “during plugin install”) to improve clarity.

Suggested change
/* Uncomment lines below to enable options for silent installation during install plugin */
/* Uncomment lines below to enable options for silent installation during plugin installation */

Copilot uses AI. Check for mistakes.
Copy link
Contributor

Choose a reason for hiding this comment

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

Updated locally to: "during plugin installation" in config.php.dist; this thread can be resolved once that patch is pushed to the branch.

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