-
Notifications
You must be signed in to change notification settings - Fork 367
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
feat(module-tools): improve logs in watch mode #4729
feat(module-tools): improve logs in watch mode #4729
Conversation
🦋 Changeset detectedLatest commit: aca5cd7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 259 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4729 +/- ##
==========================================
- Coverage 57.59% 50.72% -6.88%
==========================================
Files 672 692 +20
Lines 17794 19256 +1462
Branches 3876 4453 +577
==========================================
- Hits 10249 9768 -481
- Misses 6934 8737 +1803
- Partials 611 751 +140 ☔ View full report in Codecov by Sentry. |
Summary
🤖 Generated by Copilot at aca5cd7
This pull request refactors and improves the logging logic of the module-tools package, using a common utility function
withLogTitle
to format the log messages. It also adds a changeset file, updates the tests, and removes some unused or redundant files. The goal is to make the log output more readable, consistent, and informative.Details
🤖 Generated by Copilot at aca5cd7
withLogTitle
utility function, which simplifies the log message and adds a title with the build type, format, and target (link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link)config
parameter to thereBuild
method of theEsbuildCompiler
class and theICompiler
interface, which is used to pass the build configuration to the method (link, link, link, link)run
function of thebuilder/index.ts
file (link)printSucceed
function and the bytes to one decimal place in theprettyBytes
function of theutils/print.ts
file, which are used to print the success message and the human-readable format of the bytes (link, link)removeTscLogTime
utility function from thebuilder/dts/tsc.ts
file and add a test case for it in thebuilder.test.ts
file, which are used to remove the time prefix from the tsc logs (link, link, link)log.test.ts
file, which is no longer needed after the refactoring of the log messages (link)Related Issue
Checklist
pnpm run change
.