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

refactor: optimize dts logs and show codeblocks of tsc diagnostics #847

Merged
merged 5 commits into from
Mar 19, 2025

Conversation

Timeless0911
Copy link
Contributor

@Timeless0911 Timeless0911 commented Mar 14, 2025

Summary

  • Do not verbose some meaningless logs of api-extractor.
  • Add [api-extractor] prefix for error when bunding dts files.
  • Output relative paths for dts files/source file to reduce length of logs.
  • Use formatDiagnosticsWithColorAndContext to show codeblocks of tsc diagnostics.

bundleless dts + build

image

bundleless dts + watch

image

bundle dts + build

image

bundle dts + watch

image

Related Links

close: #631

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@Timeless0911 Timeless0911 changed the title refactor: optimize dts logs refactor: optimize dts logs and show codeblocks of tsc diagnostics Mar 14, 2025
Copy link

netlify bot commented Mar 14, 2025

Deploy Preview for rslib ready!

Name Link
🔨 Latest commit 7de4c50
🔍 Latest deploy log https://app.netlify.com/sites/rslib/deploys/67d6a8420b286f000884133e
😎 Deploy Preview https://deploy-preview-847--rslib.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Timeless0911 Timeless0911 requested a review from fi3ework March 16, 2025 10:17
@Timeless0911 Timeless0911 enabled auto-merge (squash) March 19, 2025 06:21
@Timeless0911 Timeless0911 merged commit 2a16350 into main Mar 19, 2025
25 of 30 checks passed
@Timeless0911 Timeless0911 deleted the feat/dts-logs branch March 19, 2025 06:25
@Timeless0911 Timeless0911 mentioned this pull request Mar 20, 2025
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.

[Enhancement]: Dts should better format diagnostics
3 participants