Skip to content

Conversation

@mxiao-cll
Copy link
Contributor

@mxiao-cll mxiao-cll commented Nov 25, 2025

Finnhub uses a single API to fetch stock prices, it returns more than 10K items.

@mxiao-cll mxiao-cll requested a review from a team as a code owner November 25, 2025 21:29
@github-actions
Copy link
Contributor

👋 mxiao-cll, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@github-actions
Copy link
Contributor

github-actions bot commented Nov 25, 2025

NPM Publishing labels 🏷️

🟢 This PR has valid version labels and will cause a patch bump.

Copy link
Contributor

@mmcallister-cll mmcallister-cll left a comment

Choose a reason for hiding this comment

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

are we worried about extra latency or something? there's no way to trim down their 10k? :P

@mxiao-cll mxiao-cll merged commit 509cb07 into main Nov 25, 2025
16 checks passed
@mxiao-cll mxiao-cll deleted the cache branch November 25, 2025 23:18
@github-actions
Copy link
Contributor

🚀 Successfully created version bump PR: #633

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants