Skip to content

Update REVIEW.md#82

Merged
clarkd merged 1 commit into
mainfrom
work/dc/deprecated
Jul 2, 2026
Merged

Update REVIEW.md#82
clarkd merged 1 commit into
mainfrom
work/dc/deprecated

Conversation

@clarkd

@clarkd clarkd commented Jul 2, 2026

Copy link
Copy Markdown
Member

📋 Summary

Updates guidance to not use deprecated streams on OOB dashboards.


🔍 Scope of change

  • Documentation only
  • Repository metadata or configuration
  • CI / automation
  • Other (please describe):

📚 Checklist

Summary by CodeRabbit

  • Documentation
    • Updated dashboard design guidelines to ensure dashboard tiles do not reference outdated data sources, improving overall dashboard quality and reliability.

@clarkd clarkd requested review from a team and andrewmumblebee July 2, 2026 13:01
@clarkd clarkd added the documentation Improvements or additions to documentation label Jul 2, 2026
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

🧩 Plugin PR Summary

ℹ️ No plugins were modified in this PR.

@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: ASSERTIVE

Plan: Pro

Run ID: 41471301-399f-466c-ad3f-ff5d06bfb37f

📥 Commits

Reviewing files that changed from the base of the PR and between 05aeeab and 77c6d82.

📒 Files selected for processing (1)
  • REVIEW.md

📝 Walkthrough

Walkthrough

Added a single guideline bullet to REVIEW.md specifying that out-of-the-box dashboard tiles must not reference streams with visibility type 'deprecated'.

Changes

REVIEW.md guideline update

Layer / File(s) Summary
Add deprecated stream guideline
REVIEW.md
Added a bullet stating dashboard tiles must not use deprecated streams (visibility type 'deprecated').
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is related to the change, but it's too generic to explain the specific REVIEW.md documentation update. Rename it to mention the new guidance, e.g. 'Disallow deprecated streams in OOB dashboard docs'.
✅ Passed checks (4 passed)
Check name Status Explanation
Description check ✅ Passed The description matches the miscellaneous-change template and includes a clear summary, scope, and checklist.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

@clarkd clarkd merged commit 45b9c92 into main Jul 2, 2026
1 check passed
@clarkd clarkd deleted the work/dc/deprecated branch July 2, 2026 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Development

Successfully merging this pull request may close these issues.

2 participants