Closed black7375 closed 2 months ago
The changes to the debug-log
package significantly enhance type safety and logging functionality. New TypeScript types ensure that logged objects are JSON-compatible, preventing unsupported types. Function signatures have been updated for clarity, accepting only valid argument types, while overloaded functions improve usability. Overall, these modifications lead to a more robust and user-friendly logging system.
File | Change Summary |
---|---|
packages/debug-log/src/index.ts |
Updated types for debugLog , jsonPrint , jsonLog , and jsonExpect functions to enhance type safety and JSON compatibility. Introduced new overloads for flexibility. Modified boxen padding in consola for improved log formatting. |
sequenceDiagram
participant User
participant DebugLog
participant JsonPrint
participant JsonLog
participant JsonExpect
User->>DebugLog: call debugLog(name)
DebugLog-->>User: log message
User->>JsonPrint: call jsonPrint(obj)
JsonPrint-->>User: print JSON
User->>JsonLog: call jsonLog(name, obj)
JsonLog-->>User: log JSON object
User->>JsonExpect: call jsonExpect(name, obj1, obj2)
JsonExpect-->>User: compare values and log results
In the world of logs, we hop and play,
With types so safe, they brighten the day.
JSON-friendly, our messages sing,
Robust and clear, oh what joy they bring!
So here's to the changes, with a bounce and a cheer,
A rabbit's delight, as we log without fear! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Triggered from https://github.com/mincho-js/mincho/pull/41 by @​black7375.
Checking if we can fast forward main
(47e2bf3b0a94503ea94b90bb6869c2bcd0f00e8a) to debug-log-print
(38d40dbb793f65e43c40bf78adcb3d86954f2d83).
Target branch (main
):
commit 47e2bf3b0a94503ea94b90bb6869c2bcd0f00e8a (HEAD -> main, origin/main, origin/HEAD)
Author: alstjr7375 <alstjr7375@daum.net>
Date: Sun Jul 21 14:24:27 2024 +0900
Docs: Apply suggestions from code review #39
Pull request (debug-log-print
):
commit 38d40dbb793f65e43c40bf78adcb3d86954f2d83 (pull_request/debug-log-print)
Author: alstjr7375 <alstjr7375@daum.net>
Date: Sun Jul 21 15:35:48 2024 +0900
Fix: DebugLog - declare type with function overload #40
It is possible to fast forward main
(47e2bf3b0a94503ea94b90bb6869c2bcd0f00e8a) to debug-log-print
(38d40dbb793f65e43c40bf78adcb3d86954f2d83). If you have write access to the target repository, you can add a comment with /fast-forward
to fast forward main
to debug-log-print
.
Triggered from https://github.com/mincho-js/mincho/pull/41 by @​black7375.
Checking if we can fast forward main
(47e2bf3b0a94503ea94b90bb6869c2bcd0f00e8a) to debug-log-print
(4975f3afe25ea520e053c25016e5384e2bac63cf).
Target branch (main
):
commit 47e2bf3b0a94503ea94b90bb6869c2bcd0f00e8a (HEAD -> main, origin/main, origin/HEAD)
Author: alstjr7375 <alstjr7375@daum.net>
Date: Sun Jul 21 14:24:27 2024 +0900
Docs: Apply suggestions from code review #39
Pull request (debug-log-print
):
commit 4975f3afe25ea520e053c25016e5384e2bac63cf (pull_request/debug-log-print)
Author: alstjr7375 <alstjr7375@daum.net>
Date: Sun Jul 21 15:46:21 2024 +0900
Fix: DebugLog - declare type with function overload #40
It is possible to fast forward main
(47e2bf3b0a94503ea94b90bb6869c2bcd0f00e8a) to debug-log-print
(4975f3afe25ea520e053c25016e5384e2bac63cf). If you have write access to the target repository, you can add a comment with /fast-forward
to fast forward main
to debug-log-print
.
/fast-forward
Triggered from https://github.com/mincho-js/mincho/pull/41#issuecomment-2241499686 by @​black7375.
Trying to fast forward main
(47e2bf3b0a94503ea94b90bb6869c2bcd0f00e8a) to debug-log-print
(4975f3afe25ea520e053c25016e5384e2bac63cf).
Target branch (main
):
commit 47e2bf3b0a94503ea94b90bb6869c2bcd0f00e8a (HEAD -> main, origin/main, origin/HEAD)
Author: alstjr7375 <alstjr7375@daum.net>
Date: Sun Jul 21 14:24:27 2024 +0900
Docs: Apply suggestions from code review #39
Pull request (debug-log-print
):
commit 4975f3afe25ea520e053c25016e5384e2bac63cf (pull_request/debug-log-print)
Author: alstjr7375 <alstjr7375@daum.net>
Date: Sun Jul 21 15:46:21 2024 +0900
Fix: DebugLog - declare type with function overload #40
Fast forwarding main
(47e2bf3b0a94503ea94b90bb6869c2bcd0f00e8a) to debug-log-print
(4975f3afe25ea520e053c25016e5384e2bac63cf).
$ git push origin 4975f3afe25ea520e053c25016e5384e2bac63cf:main
To https://github.com/mincho-js/mincho.git
47e2bf3..4975f3a 4975f3afe25ea520e053c25016e5384e2bac63cf -> main
Description
Adjust the type and print of
@mincho/debug-log
pacakge.Related Issue
40
Summary by CodeRabbit
New Features
debugLog
,jsonPrint
,jsonLog
, andjsonExpect
functions with improved signatures for flexibility.Improvements
jsonExpect
function, improving usability.