Skip to content

Refactored logger to add context#2695

Merged
dpwatrous merged 2 commits intomainfrom
dpwatrous/vnext-logger
Mar 27, 2023
Merged

Refactored logger to add context#2695
dpwatrous merged 2 commits intomainfrom
dpwatrous/vnext-logger

Conversation

@dpwatrous
Copy link
Member

This should enable us to create a logger for the portal.

Also added a fake clock for controlling how time advances manually.

@dpwatrous dpwatrous force-pushed the dpwatrous/vnext-logger branch 4 times, most recently from 979f4f9 to 50cf03a Compare March 27, 2023 13:30
This should enable us to create a logger for the portal
@dpwatrous dpwatrous force-pushed the dpwatrous/vnext-logger branch from 50cf03a to 35e5aae Compare March 27, 2023 13:40
@codecov
Copy link

codecov bot commented Mar 27, 2023

Codecov Report

Merging #2695 (d01a1f7) into main (3d905c9) will increase coverage by 0.05%.
The diff coverage is 75.42%.

❗ Current head d01a1f7 differs from pull request most recent head 35e5aae. Consider uploading reports for the commit 35e5aae to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2695      +/-   ##
==========================================
+ Coverage   66.18%   66.24%   +0.05%     
==========================================
  Files        1200     1204       +4     
  Lines       33375    33453      +78     
  Branches     6089     6100      +11     
==========================================
+ Hits        22090    22161      +71     
- Misses      11154    11157       +3     
- Partials      131      135       +4     
Impacted Files Coverage Δ
packages/common/src/ui-common/action/action.ts 81.53% <0.00%> (ø)
...es/react/src/ui-react/components/form/checkbox.tsx 70.00% <0.00%> (ø)
packages/react/src/ui-react/theme/theme-util.ts 34.78% <0.00%> (ø)
...ice/src/ui-service/certificate/certificate-util.ts 75.00% <0.00%> (ø)
...d/src/ui-playground/demo/form/action-form-demo.tsx 22.22% <33.33%> (-0.32%) ⬇️
.../src/ui-service/certificate/certificate-service.ts 50.00% <33.33%> (+1.51%) ⬆️
...es/common/src/ui-common/datetime/standard-clock.ts 50.00% <50.00%> (ø)
packages/common/src/ui-common/form/parameter.ts 88.54% <50.00%> (+0.12%) ⬆️
...es/common/src/ui-common/logging/abstract-logger.ts 67.64% <65.62%> (-32.36%) ⬇️
.../src/ui-common/environment/abstract-environment.ts 78.33% <66.66%> (-1.67%) ⬇️
... and 10 more

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3d905c9...35e5aae. Read the comment docs.

@dpwatrous dpwatrous enabled auto-merge March 27, 2023 14:29
@dpwatrous dpwatrous merged commit 44f79c1 into main Mar 27, 2023
@dpwatrous dpwatrous deleted the dpwatrous/vnext-logger branch March 27, 2023 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants