Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WASP v1.50.5 changeset #1548

Merged
merged 1 commit into from
Jan 10, 2025
Merged

WASP v1.50.5 changeset #1548

merged 1 commit into from
Jan 10, 2025

Conversation

Tofel
Copy link
Contributor

@Tofel Tofel commented Jan 10, 2025


Below is a summarization created by an LLM (gpt-4-0125-preview). Be mindful of hallucinations and verify accuracy.

Why

The changes improve BenchSpy, a performance comparison tool, by adding threshold validation, ensuring reports are not nil, and enhancing readability when there's an infinite metric change. Additionally, error handling is simplified by returning a single error instead of a map of errors, making the tool more user-friendly and robust.

What

  • wasp/.changeset/v1.50.5.md
    • Added details about improvements to BenchSpy, including threshold validation, nil report validation, and better readability for infinite metric change. This makes the tool more reliable and easier to use.
    • Modified the CompareDirectWithThresholds function to return a single error instead of map[string][]error, simplifying the error handling process.

@Tofel Tofel marked this pull request as ready for review January 10, 2025 15:23
@Tofel Tofel requested review from sebawo and a team as code owners January 10, 2025 15:23
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@Tofel Tofel merged commit 63fcb37 into main Jan 10, 2025
55 of 56 checks passed
@Tofel Tofel deleted the wasp-v1.50.5-changeset branch January 10, 2025 15:54
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.

2 participants