Skip to content

Conversation

@sunker
Copy link
Contributor

@sunker sunker commented Oct 24, 2025

What this PR does / why we need it:

Follow-up from #2241 (comment). Also removing another section in this doc where the Saga website is being referenced.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

@Copilot Copilot AI review requested due to automatic review settings October 24, 2025 12:22
@sunker sunker requested review from a team as code owners October 24, 2025 12:22
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 removes references to the Saga Design System from the plugin tools documentation. The Saga Design System link is being removed from two locations in the best practices documentation where it was previously referenced alongside Grafana UI components guidance.

Key changes:

  • Removed Saga Design System reference and link from the frontend best practices section
  • Removed entire bullet point about practicing good front-end design that referenced Saga
  • Retained guidance about using Grafana UI components and their documentation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 24, 2025

Hello! 👋 This repository uses Auto for releasing packages using PR labels.

❌ This PR cannot be merged until the following issues are addressed:

  • This PR is missing one of the following labels: patch, minor, major, no-changelog.
  • Optionally if using a patch, minor or major label also add the release label if you would like this PR to trigger npm package publishing.
🏷️ More info about which labels to use
  • If the changes only affect the docs website, documentation, or this repository's tooling add the no-changelog label.
  • If there are changes to any of the npm packages src files please choose from one of the following labels:
    • 🐛 if this PR fixes a bug add the patch label
    • 🚀 if this PR includes an enhancement add the minor label
    • 💥 if this PR includes a breaking change add the major label
  • Optionally, if you would like this PR to publish new versions of packages when it is merged add the release label.

@sunker sunker requested a review from urbiz-grafana October 24, 2025 12:23
@sunker sunker self-assigned this Oct 24, 2025
@sunker sunker moved this from 📬 Triage to 🔬 In review in Plugins Platform / Grafana Community Oct 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🔬 In review

Development

Successfully merging this pull request may close these issues.

3 participants