Skip to content

Add more detailed bundling error descriptions #5907

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dmerand
Copy link
Contributor

@dmerand dmerand commented May 27, 2025

WHY are these changes introduced?

The fix addresses issue #2092 by ensuring build errors are always visible to developers, helping them quickly identify and resolve issues in their extension code.

How to test your changes?

This upstack draft PR has testing instructions. Once this PR is approved, I'll close that draft.

Measuring impact

How do we know this change was effective? Please choose one:

  • n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes

Copy link
Contributor Author

dmerand commented May 27, 2025

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

Copy link
Contributor

github-actions bot commented May 27, 2025

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements
78.02% (+0.02% 🔼)
12463/15975
🟡 Branches
72.23% (+0.09% 🔼)
6045/8369
🟡 Functions 78.28% 3272/4180
🟡 Lines
78.45% (+0.02% 🔼)
11791/15030

Test suite run success

2867 tests passing in 1253 suites.

Report generated by 🧪jest coverage report action from cd41af8

@dmerand dmerand force-pushed the dlm-fix-bundling-errors branch 5 times, most recently from 33914d6 to c32ba32 Compare May 28, 2025 19:13
@dmerand dmerand marked this pull request as ready for review May 29, 2025 15:57
@dmerand dmerand requested a review from a team as a code owner May 29, 2025 15:57

This comment has been minimized.

@dmerand dmerand force-pushed the dlm-fix-bundling-errors branch from c32ba32 to cd41af8 Compare May 29, 2025 16:05
@dmerand dmerand requested a review from a team as a code owner May 29, 2025 16:05
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.

1 participant