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

Performance test fixes and optimizations #159

Merged
merged 5 commits into from
Dec 18, 2023
Merged

Performance test fixes and optimizations #159

merged 5 commits into from
Dec 18, 2023

Conversation

Siegrift
Copy link
Collaborator

@Siegrift Siegrift commented Dec 6, 2023

Closes #158

Rationale

I cherry picked various fixes from #154 and opening a separate PR. Review commit by commit.

@Siegrift Siegrift self-assigned this Dec 6, 2023
@Siegrift Siegrift force-pushed the validate-signed-api-name branch 2 times, most recently from f1af37f to 93ca962 Compare December 6, 2023 16:01
Base automatically changed from validate-signed-api-name to main December 6, 2023 16:27
@Siegrift Siegrift force-pushed the perf-test-fixes branch 2 times, most recently from 8aa7421 to a00c6c1 Compare December 6, 2023 16:45
@Siegrift Siegrift changed the title WIP: Performance test fixes and optimizations Performance test fixes and optimizations Dec 11, 2023
@Siegrift Siegrift changed the title Performance test fixes and optimizations TODO: Performance test fixes and optimizations Dec 11, 2023
@Siegrift Siegrift requested a review from andreogle December 12, 2023 06:31
@Siegrift Siegrift changed the title TODO: Performance test fixes and optimizations Performance test fixes and optimizations Dec 12, 2023
Copy link
Member

@andreogle andreogle left a comment

Choose a reason for hiding this comment

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

👍🏻

packages/api/src/server.ts Outdated Show resolved Hide resolved
@Siegrift Siegrift merged commit 14eec89 into main Dec 18, 2023
4 checks passed
@Siegrift Siegrift deleted the perf-test-fixes branch December 18, 2023 20:29
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.

Push data immediately as they are fetched
3 participants