Skip to content

Conversation

@danielreynolds1
Copy link
Contributor

@danielreynolds1 danielreynolds1 commented Feb 4, 2025

Summary of the changes (Less than 80 chars)

  • [Fusion] Added post-merge validation rule "ReferenceToInaccessibleTypeRule".

@danielreynolds1 danielreynolds1 force-pushed the dr/input-field-references-inaccessible-type-rule branch from 75059e3 to abeddfd Compare February 4, 2025 21:40
@codecov
Copy link

codecov bot commented Feb 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (860a87c) to head (ca6a694).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@     Coverage Diff      @@
##   main   #7992   +/-   ##
============================
============================

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@glen-84 glen-84 added the ✋blocked The issue is blocked by an external dependency. label Feb 6, 2025
@glen-84 glen-84 changed the title [Fusion] Added post-merge validation rule InputFieldReferencesInaccessibleTypeRule [Fusion] Added post-merge validation rule "InputFieldReferencesInaccessibleTypeRule" Feb 13, 2025
@glen-84 glen-84 changed the title [Fusion] Added post-merge validation rule "InputFieldReferencesInaccessibleTypeRule" [Fusion] Added post-merge validation rule "ReferenceToInaccessibleTypeRule" Nov 3, 2025
@glen-84 glen-84 removed the ✋blocked The issue is blocked by an external dependency. label Nov 3, 2025
@glen-84 glen-84 marked this pull request as ready for review November 3, 2025 12:04
@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

🚀 Performance Test Results

📊 Response Time

Current

Test Min Med Max Avg P90 P95 P99
Single Fetch 0.76ms 1.54ms 15.84ms 2.06ms 3.35ms 4.43ms 9.29ms
DataLoader 1.67ms 3.13ms 70.45ms 4.35ms 7.90ms 11.35ms 18.64ms

Baseline

Test Min Med Max Avg P90 P95 P99
Single Fetch 0.76ms 1.64ms 22.30ms 2.06ms 3.20ms 4.24ms 8.83ms
DataLoader 1.66ms 3.07ms 56.38ms 3.80ms 6.23ms 8.34ms 14.72ms

Change vs Baseline

Test Min Med Max Avg P90 P95 P99
Single Fetch ✅ (0.11% worse) 🎉 (6.03% better) 🎉 (28.96% better) ✅ (0.31% better) ✅ (4.70% worse) ✅ (4.51% worse) ⚠️ (5.21% worse)
DataLoader ✅ (0.52% worse) ✅ (1.87% worse) ⚠️ (24.96% worse) ⚠️ (14.45% worse) ⚠️ (26.82% worse) ⚠️ (36.20% worse) ⚠️ (26.66% worse)

⚡ Throughput

Test Metric Current Baseline Change
Single Fetch Requests/sec 394.49 req/s 394.51 req/s ✅ (0.00% worse)
DataLoader Requests/sec 393.41 req/s 393.53 req/s ✅ (0.03% worse)

🎯 Reliability

Test Error Rate
Single Fetch 0.00% ✅
DataLoader 0.00% ✅

🔍 Analysis

⚠️ Performance regression detected. Response times have increased significantly compared to baseline.


Run 19034046020 • Commit b77ada5 • Mon, 03 Nov 2025 12:20:07 GMT

@glen-84 glen-84 merged commit 23758e8 into main Nov 3, 2025
113 checks passed
@glen-84 glen-84 deleted the dr/input-field-references-inaccessible-type-rule branch November 3, 2025 12:23
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