fix(utils): improve ANSI handling and logging debug information#720
fix(utils): improve ANSI handling and logging debug information#720Kaibalya-Mohanty wants to merge 7 commits into
Conversation
Signed-off-by: KAIBALYA MOHANTY <168870673+Kaibalya-Mohanty@users.noreply.github.com>
Signed-off-by: KAIBALYA MOHANTY <168870673+Kaibalya-Mohanty@users.noreply.github.com>
Signed-off-by: KAIBALYA MOHANTY <168870673+Kaibalya-Mohanty@users.noreply.github.com>
Signed-off-by: KAIBALYA MOHANTY <168870673+Kaibalya-Mohanty@users.noreply.github.com>
- Improved ANSI escape sequence handling with robust regex - Added type safety for non-string inputs - Enhanced logging with debug statements for better traceability Signed-off-by: KAIBALYA MOHANTY <168870673+Kaibalya-Mohanty@users.noreply.github.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Kaibalya-Mohanty
changed the title
|
hi @Kaibalya-Mohanty I believe this PR should not have the DocString changes from #714 |
|
@ada333 Thanks for pointing this out! |
Updated docstrings to be more concise while retaining essential information. Signed-off-by: KAIBALYA MOHANTY <168870673+Kaibalya-Mohanty@users.noreply.github.com>
|
@Kaibalya-Mohanty as both are doc improvements, can you please combine both changes on #704 ? |
3 participants
Summary
This PR improves utility functions in kale/utils.py.
Changes
Motivation
This change improves log readability and debugging experience.
Impact