pepperplus/pepperplus-cb
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Restructure logging #21

Merged
k.marinus merged 11 commits from refactor/logging into dev 2025-11-05 15:31:06 +00:00
Conversation 7 Commits 11 Files Changed 34 +451 -411
k.marinus commented 2025-11-05 14:14:40 +00:00 (Migrated from git.science.uu.nl)
Copy Link

Standardized logging across the entire codebase. Implemented a logging config file .logging_config.yaml, which should hold the entire config.

Several new logging levels are available (OBSERVATION and ACTION), intended for easier separation of log messages in the UI.

Also implemented sending logging to the UI through an SSE endpoint (/logs/stream).

To verify, in addition to the usual "code runs" and "tests pass", make sure that logging looks logical and self-consistent throughout the codebase.

ref: N25B-233

Standardized logging across the entire codebase. Implemented a logging config file `.logging_config.yaml`, which should hold the entire config. Several new logging levels are available (OBSERVATION and ACTION), intended for easier separation of log messages in the UI. Also implemented sending logging to the UI through an SSE endpoint (`/logs/stream`). To verify, in addition to the usual "code runs" and "tests pass", make sure that logging looks logical and self-consistent throughout the codebase. ref: N25B-233
k.marinus commented 2025-11-05 14:14:40 +00:00 (Migrated from git.science.uu.nl)
Copy Link

requested review from @8464960

requested review from @8464960
k.marinus commented 2025-11-05 15:10:08 +00:00 (Migrated from git.science.uu.nl)
Copy Link

changed title from feat: add colored and formatted logging to Restructure logging

<p>changed title from <code class="idiff"><span class="idiff left right deletion">feat: add colored and formatted</span> logging</code> to <code class="idiff"><span class="idiff left right addition">Restructure</span> logging</code></p>
k.marinus commented 2025-11-05 15:13:01 +00:00 (Migrated from git.science.uu.nl)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>9e711948 - fix: tests pass</li></ul> [Compare with previous version](/ics/sp/2025/n25b/pepperplus-cb/-/merge_requests/21/diffs?diff_id=133272&start_sha=d9fef2209056fb7d9a3c8b90a2606d0e3b568f4e)
8464960 commented 2025-11-05 15:28:08 +00:00 (Migrated from git.science.uu.nl)
Copy Link

approved this merge request

approved this merge request
8464960 commented 2025-11-05 15:31:02 +00:00 (Migrated from git.science.uu.nl)
Copy Link

Nice job!

Nice job!
8464960 commented 2025-11-05 15:31:06 +00:00 (Migrated from git.science.uu.nl)
Copy Link

mentioned in commit 6235fcdaf4

mentioned in commit 6235fcdaf42f6f96ed024ef95311be4b1a8735b8
8464960 (Migrated from git.science.uu.nl) merged commit 6235fcdaf4 into dev 2025-11-05 15:31:06 +00:00
8464960 (Migrated from git.science.uu.nl) approved these changes 2026-02-02 13:29:06 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
8464960
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
kasper twirre pepperplus
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: pepperplus/pepperplus-cb#21
Delete Branch "refactor/logging"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?