Skip to content

Conversation

@brandon-pereira
Copy link
Member

matches other tests in this file

@brandon-pereira brandon-pereira requested review from a team and wrn14897 and removed request for a team December 30, 2025 20:38
@changeset-bot
Copy link

changeset-bot bot commented Dec 30, 2025

⚠️ No Changeset found

Latest commit: 74d2f61

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Dec 30, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
hyperdx-v2-oss-app Ready Ready Preview, Comment Dec 30, 2025 8:38pm

@claude
Copy link

claude bot commented Dec 30, 2025

✅ No critical issues found.

The change is consistent with existing patterns in the test file - other getChartContainers() assertions already use timeout: 10000 (lines 44, 211). Good catch on the flaky test.

@github-actions
Copy link
Contributor

E2E Test Results

All tests passed • 52 passed • 5 skipped • 689s

Status Count
✅ Passed 52
❌ Failed 0
⚠️ Flaky 0
⏭️ Skipped 5

Tests ran across 4 shards in parallel.

View full report →

@kodiakhq kodiakhq bot merged commit f1cac57 into main Dec 30, 2025
14 checks passed
@kodiakhq kodiakhq bot deleted the brandon/fix-flakey-dashboard-test branch December 30, 2025 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants