Skip to content

Conversation

@Timeless0911
Copy link
Contributor

@Timeless0911 Timeless0911 commented Jan 8, 2026

Summary

Update onAfterBuild hook in createRslib to check for compilation errors before logging the success message in watch mode of fist compile.

If errors exist, log an error message instead to avoid misleading "build complete" status when errors occurred.

Dts errors are not include in the stats, it is currently difficult to achieve.

before

image

after

image

Related Links

Checklist

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

Copilot AI review requested due to automatic review settings January 8, 2026 09:12
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the build status logging in watch mode by checking for compilation errors before logging the success message. When errors are present, it now logs an error message instead of a misleading success message.

  • Added error detection logic to the onAfterBuild hook using stats?.hasErrors()
  • Changed log level from success to error when build errors occur
  • Updated message consistency (changed "build complete" to "build completed")

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Timeless0911 Timeless0911 merged commit 25e8d48 into main Jan 8, 2026
16 checks passed
@Timeless0911 Timeless0911 deleted the feat/improve-watch-mode-logging branch January 8, 2026 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants