Skip to content

Conversation

@impoiler
Copy link
Contributor

No description provided.

This was referenced Oct 28, 2025
Copy link
Contributor Author

impoiler commented Oct 28, 2025

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 28, 2025

📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Added a comprehensive guide for tagging test runs in the Via UI, including instructions for creating and selecting tags and filtering tagged runs within the runs table.

Walkthrough

Added a new documentation page "offline-evals/via-ui/advanced/tag-reports.mdx" describing how to tag and filter test runs in the Via UI Advanced section, and updated docs.json to include the new page under Via UI > Advanced navigation.

Changes

Cohort / File(s) Summary
Documentation navigation update
docs.json
Added a navigation entry for offline-evals/via-ui/advanced/tag-reports under the Via UI > Advanced group.
New tagging guide (MDX)
offline-evals/via-ui/advanced/tag-reports.mdx
New page "Tag test runs" with two-step workflow: (1) add tags via Add tags button and dropdown (includes image), (2) filter runs by tag from the Runs page (includes image).

Sequence Diagram(s)

N/A — documentation-only changes; no control-flow or runtime behavior modified.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Verify docs.json entry matches existing navigation schema and ordering.
  • Confirm image paths in tag-reports.mdx resolve and render.
  • Quick technical proofreading for accuracy and clarity.

Poem

🐰 I hopped through docs with a pen so bright,
Tags for runs now in clear daylight—
Click, pick, and filter with a cheer,
Finding results is swift and near! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess relevance to the changeset. Add a brief description explaining the purpose and scope of the documentation additions, such as why this documentation is needed and what it helps users accomplish.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title '[Maxim] Test run report tags docs' clearly identifies the main change: addition of documentation for test run report tags functionality.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch improvement/tag-runs-docs

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 41592f7 and a3d241c.

⛔ Files ignored due to path filters (2)
  • images/docs/evaluate/how-to/optimize-evaluation-processes/tag-reports/add-tags-to-report.png is excluded by !**/*.png
  • images/docs/evaluate/how-to/optimize-evaluation-processes/tag-reports/filter-reports-by-tag.png is excluded by !**/*.png
📒 Files selected for processing (2)
  • docs.json (1 hunks)
  • offline-evals/via-ui/advanced/tag-reports.mdx (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • offline-evals/via-ui/advanced/tag-reports.mdx
🔇 Additional comments (1)
docs.json (1)

78-78: Documentation reference correctly added to navigation.

The new page reference is properly formatted, positioned logically within the Advanced section, and follows the existing path convention. The entry integrates cleanly into the navigation structure.


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

@impoiler impoiler marked this pull request as ready for review November 3, 2025 13:02
@impoiler impoiler force-pushed the improvement/tag-runs-docs branch from aef2bd1 to d37bfe0 Compare November 3, 2025 13:06
@impoiler impoiler force-pushed the improvement/human-eval-docs branch from ffc291e to ae84dc4 Compare November 3, 2025 13:07
@impoiler impoiler force-pushed the improvement/tag-runs-docs branch from d37bfe0 to b918a79 Compare November 3, 2025 13:09
@impoiler impoiler force-pushed the improvement/human-eval-docs branch 2 times, most recently from b500eac to 26caf24 Compare November 3, 2025 14:51
@impoiler impoiler force-pushed the improvement/tag-runs-docs branch from b918a79 to 4270cd6 Compare November 3, 2025 14:51
coderabbitai[bot]
coderabbitai bot previously approved these changes Nov 3, 2025
@impoiler impoiler force-pushed the improvement/human-eval-docs branch from 26caf24 to 6aa9c1b Compare November 3, 2025 15:19
@impoiler impoiler force-pushed the improvement/tag-runs-docs branch from 4270cd6 to 41592f7 Compare November 3, 2025 15:19
Copy link
Contributor Author

impoiler commented Nov 3, 2025

Merge activity

@akshaydeo akshaydeo changed the base branch from improvement/human-eval-docs to graphite-base/158 November 5, 2025 14:56
@akshaydeo akshaydeo force-pushed the improvement/tag-runs-docs branch from 41592f7 to c4c4bc9 Compare November 5, 2025 14:58
@graphite-app graphite-app bot changed the base branch from graphite-base/158 to main November 5, 2025 14:58
@graphite-app graphite-app bot dismissed coderabbitai[bot]’s stale review November 5, 2025 14:58

The base branch was changed.

@akshaydeo akshaydeo force-pushed the improvement/tag-runs-docs branch from c4c4bc9 to a3d241c Compare November 5, 2025 14:58
@akshaydeo akshaydeo merged commit c8b912e into main Nov 5, 2025
3 checks passed
@akshaydeo akshaydeo deleted the improvement/tag-runs-docs branch November 5, 2025 15:08
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