Skip to content

Add badge for Grafana integration test#558

Merged
amotl merged 2 commits intomainfrom
badge-grafana
Mar 3, 2026
Merged

Add badge for Grafana integration test#558
amotl merged 2 commits intomainfrom
badge-grafana

Conversation

@amotl
Copy link
Member

@amotl amotl commented Mar 3, 2026

@coderabbitai
Copy link

coderabbitai bot commented Mar 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 43eb441 and 9047031.

📒 Files selected for processing (3)
  • docs/integrate/status.md
  • docs/integrate/superset/index.md
  • docs/integrate/superset/usage.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • docs/integrate/status.md

Walkthrough

Added a Grafana entry to the Integrations "Applications" table and updated several Superset doc links to use the new user-docs paths; all changes are content-only edits to markdown files (no code or control-flow changes).

Changes

Cohort / File(s) Summary
Integrations status
docs/integrate/status.md
Inserted a Grafana anchor/workflow badge row into the Applications table.
Superset docs — index
docs/integrate/superset/index.md
Rewrote two external Superset links to point to user-docs paths instead of docs paths.
Superset docs — usage
docs/integrate/superset/usage.md
Updated two external Superset links to user-docs targets (connect/register DB links).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

cross linking, maintenance

Suggested reviewers

  • surister
  • karynzv
  • hammerhead

Poem

🐰 A Grafana badge hops into view,
Links tidy and trimmed, all shiny and new,
Docs bounce along with a cheerful tune,
Dashboards and guides under the moon.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: adding a Grafana integration badge to the status documentation, which aligns with the primary modification in docs/integrate/status.md.
Description check ✅ Passed The description is related to the changeset, mentioning the addition of a badge for another integration test (Grafana) and providing a preview link, though it doesn't detail the URL updates in other files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch badge-grafana

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@amotl amotl marked this pull request as ready for review March 3, 2026 05:27
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@docs/integrate/status.md`:
- Line 48: The Grafana badge image tag is missing alt text, causing
accessibility lint MD045; update the <img> element (the Grafana badge HTML tag)
to include a descriptive alt attribute (for example alt="Grafana CI status" or
similar) so the image becomes accessible to non-visual readers and satisfies the
markdown lint rule.

ℹ️ Review info

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c755401 and 43eb441.

📒 Files selected for processing (1)
  • docs/integrate/status.md

@amotl amotl requested review from matriv and seut March 3, 2026 06:50
@amotl amotl added cross linking Linking to different locations of the documentation. maintenance C'est la vie. labels Mar 3, 2026
@amotl amotl merged commit 0488d9e into main Mar 3, 2026
3 checks passed
@amotl amotl deleted the badge-grafana branch March 3, 2026 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cross linking Linking to different locations of the documentation. maintenance C'est la vie.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants