Skip to content

Conversation

@alexcambose
Copy link
Contributor

Summary

Rationale

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

@alexcambose alexcambose requested a review from cprussin August 18, 2025 14:31
@alexcambose alexcambose self-assigned this Aug 18, 2025
@alexcambose alexcambose requested a review from a team as a code owner August 18, 2025 14:31
@vercel
Copy link

vercel bot commented Aug 18, 2025

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

Project Deployment Preview Comments Updated (UTC)
component-library Ready Ready Preview Comment Aug 18, 2025 6:01pm
insights Ready Ready Preview Comment Aug 18, 2025 6:01pm
5 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
api-reference Skipped Skipped Aug 18, 2025 6:01pm
developer-hub Skipped Skipped Aug 18, 2025 6:01pm
entropy-explorer Skipped Skipped Aug 18, 2025 6:01pm
proposals Skipped Skipped Aug 18, 2025 6:01pm
staking Skipped Skipped Aug 18, 2025 6:01pm

@vercel vercel bot temporarily deployed to Preview – entropy-explorer August 18, 2025 14:34 Inactive
@vercel vercel bot temporarily deployed to Preview – staking August 18, 2025 14:34 Inactive
@vercel vercel bot temporarily deployed to Preview – api-reference August 18, 2025 14:34 Inactive
@vercel vercel bot temporarily deployed to Preview – developer-hub August 18, 2025 14:34 Inactive
@vercel vercel bot temporarily deployed to Preview – proposals August 18, 2025 14:34 Inactive
Copy link
Collaborator

@cprussin cprussin left a comment

Choose a reason for hiding this comment

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

I think if you simplify this function a bit it will result in better errors and code which is easier to understand

@vercel vercel bot temporarily deployed to Preview – developer-hub August 18, 2025 15:48 Inactive
@vercel vercel bot temporarily deployed to Preview – proposals August 18, 2025 15:48 Inactive
@vercel vercel bot temporarily deployed to Preview – entropy-explorer August 18, 2025 15:48 Inactive
@vercel vercel bot temporarily deployed to Preview – api-reference August 18, 2025 15:48 Inactive
@vercel vercel bot temporarily deployed to Preview – staking August 18, 2025 15:48 Inactive
@vercel vercel bot temporarily deployed to Preview – developer-hub August 18, 2025 17:58 Inactive
@vercel vercel bot temporarily deployed to Preview – entropy-explorer August 18, 2025 17:58 Inactive
@vercel vercel bot temporarily deployed to Preview – proposals August 18, 2025 17:58 Inactive
@vercel vercel bot temporarily deployed to Preview – staking August 18, 2025 17:58 Inactive
@vercel vercel bot temporarily deployed to Preview – api-reference August 18, 2025 17:58 Inactive
@alexcambose alexcambose merged commit e78fec7 into main Aug 18, 2025
9 of 10 checks passed
@alexcambose alexcambose deleted the fix/headers-invariant branch August 18, 2025 18:00
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