Skip to content
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.

Sign up for GitHub

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

Jump to bottom

BADGERS-307 feat(debugger): Record changes to metric values over time #324

Merged
merged 133 commits into from
Mar 7, 2024
Merged

BADGERS-307 feat(debugger): Record changes to metric values over time #324

merged 133 commits into from
Mar 7, 2024

Conversation

eirikbjornr
Copy link
Contributor

📺 What

Enhance logging capabilities to provide a structured logging system.

🛠 How

  • TypeScript
  • Refactor the on-screen logger
  • Change Chronicle data structure to a set structure with semantic categories:
    • Messages: String log messages
    • Metrics: Values that change over time
    • Traces: One-off values with no history

ShiningTrapez and others added 30 commits January 4, 2024 08:38
@ShiningTrapez
Linting on ReadyHelper
718a492
@ShiningTrapez
What doesn't this change
cf51217
@ShiningTrapez
Remove Terser - Unused
703d56b
@eirikbjornr
it aint much
1d22403
@eirikbjornr
but it's darn honest work
087c374
@eirikbjornr
wip: add media element types
0dd7c8c
@ShiningTrapez
Combine Message and Metric Types
e77b7e6
@ShiningTrapez
[WIP] Classify Chronicle
bed3809
@ShiningTrapez
Fix Chronicle Unit Tests
956fbd7
@ShiningTrapez
Fix all tests
a3f1cc6
@eirikbjornr
chore: change chronicle to TypeScript
990bee8
@eirikbjornr
wip: type bonanza
29e251d
@eirikbjornr
wip: metric interface
d9364de
@ShiningTrapez
Type that extracts Metric Value based on key
356f6fa
@ShiningTrapez
Fix Types for Metric Get
7d3f90c
@eirikbjornr
wip: use new api
b74c751
@ShiningTrapez
WIP Message Push Functions
22dc014
@ShiningTrapez
Remove API Call and Event from Trace
5a1aaa9
@ShiningTrapez
Export EntryForIdentifier, instead of Message/MetricForKey/Level
cb7f07d
@eirikbjornr
wip: elmo burn emoji
1da1693
@ShiningTrapez
getLatestMetric
ad73894
@eirikbjornr
refine types
c590006
@eirikbjornr
chore: disable prefer-at
5051460
@eirikbjornr
feat: chronicle is amazing now
48bc0ad
@ShiningTrapez
DebugPresenter => DebugFormatter
156e2b9
@ShiningTrapez
Minor linting changes
195293b
@eirikbjornr
feat: add event trace
7495333
@eirikbjornr
chore: make debugtool TypeScript
a5a27ac
@eirikbjornr
chore: make debugformatter TypeScript
b58a5eb
@eirikbjornr
wip: im bad at splitting commits
2683459
@eirikbjornr eirikbjornr added the semver minor This PR is a semver minor release label Mar 5, 2024
@eirikbjornr eirikbjornr requested a review from a team as a code owner March 5, 2024 11:23
@eirikbjornr eirikbjornr merged commit 9bc1663 into master Mar 7, 2024
4 checks passed
@eirikbjornr eirikbjornr deleted the BADGERS-307 branch March 7, 2024 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jlks jlks jlks approved these changes

Assignees

@ShiningTrapez ShiningTrapez

@eirikbjornr eirikbjornr

Labels
semver minor This PR is a semver minor release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@eirikbjornr @jlks @ShiningTrapez @jamesbungay-bbc