Skip to content
New issue

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

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

Already on GitHub? # to your account

[DevOverlay] fix total error count #75500

Merged
merged 1 commit into from
Feb 3, 2025

Conversation

ztanner
Copy link
Member

@ztanner ztanner commented Jan 31, 2025

readyErrors seems to be the value that we're using to actually display the errors in the overlay, while state.errors seems to often contain duplicates. This causes a drift in the number of errors reported in the overlay vs the indicator. We should use the same source of truth for both.

@ijjk ijjk added created-by: Next.js team PRs by the Next.js team. type: next labels Jan 31, 2025
Copy link
Member Author

ztanner commented Jan 31, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ztanner ztanner marked this pull request as ready for review January 31, 2025 03:14
Copy link
Member

@devjiwonchoi devjiwonchoi left a comment

Choose a reason for hiding this comment

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

Good catch, thanks!

@ztanner ztanner merged commit 309092f into canary Feb 3, 2025
140 of 152 checks passed
@ztanner ztanner deleted the 01-30-_devoverlay_fix_total_error_count branch February 3, 2025 15:20
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 18, 2025
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants