Skip to content

Fix docs - #1741

Merged
jertel merged 3 commits into
jertel:masterfrom
nsano-rururu:fix_docs
Dec 1, 2025
Merged

Fix docs#1741
jertel merged 3 commits into
jertel:masterfrom
nsano-rururu:fix_docs

Conversation

@nsano-rururu

@nsano-rururu nsano-rururu commented Dec 1, 2025

Copy link
Copy Markdown
Collaborator

Description

Checklist

  • I have reviewed the contributing guidelines.
  • I have included unit tests for my changes or additions.
  • I have successfully run make test-docker with my changes.
  • I have manually tested all relevant modes of the change in this PR.
  • I have updated the documentation.
  • I have updated the changelog.

Questions or Comments

Copilot AI review requested due to automatic review settings December 1, 2025 15:45

Copilot AI left a comment

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.

Pull request overview

This PR updates the documentation to reflect accurate alerter names and current URLs. The changes improve consistency between the alerter list in elastalert.rst and the detailed alerter sections in alerts.rst.

  • Updates alerter names to match their actual implementations (e.g., "Line Notify" → "LINE Messaging API", "Grafana OnCall" → "Grafana IRM")
  • Updates URLs to point to current documentation (Alertmanager and Grafana IRM)
  • Adds consistent line numbering and removes trailing whitespace

Reviewed changes

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

File Description
docs/source/elastalert.rst Updates alerter list with corrected names (Grafana IRM, IRIS, LINE Messaging API, Webex Webhook, YZJ) and improves formatting with consistent line numbering
docs/source/alerts.rst Updates section titles and URLs for Alertmanager and Grafana IRM alerters to point to current documentation

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

jertel
jertel previously approved these changes Dec 1, 2025
@jertel
jertel merged commit 30115fa into jertel:master Dec 1, 2025
1 check passed
@nsano-rururu
nsano-rururu deleted the fix_docs branch December 1, 2025 16:19
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