Skip to content

fix: ctx logger paddingMessage should be a getter - #86

Merged
fengmk2 merged 2 commits into
masterfrom
fix-paddingMessage
Jan 20, 2023
Merged

fengmk2 merged 2 commits into
masterfrom
fix-paddingMessage

Conversation

@fengmk2

@fengmk2 fengmk2 commented Jan 20, 2023 •

Copy link
Copy Markdown
Member

add missing ctx define on EggContextLogger and LoggerMeta

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)
Description of change

add missing ctx define on EggContextLogger
@codecov

codecov Bot commented Jan 20, 2023 •

Copy link
Copy Markdown

Codecov Report

Base: 98.69% // Head: 98.69% // No change to project coverage 👍

Coverage data is based on head (7b2a1f8) compared to base (65f3bff).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #86   +/-   ##
=======================================
  Coverage   98.69%   98.69%           
=======================================
  Files          14       14           
  Lines        1300     1300           
  Branches      214      214           
=======================================
  Hits         1283     1283           
  Misses         17       17           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@fengmk2
fengmk2 merged commit 5417064 into master Jan 20, 2023
@fengmk2
fengmk2 deleted the fix-paddingMessage branch January 20, 2023 13:46
fengmk2 pushed a commit that referenced this pull request Jan 20, 2023
[skip ci]

## [3.2.1](v3.2.0...v3.2.1) (2023-01-20)

### Bug Fixes

* ctx logger paddingMessage should be a getter ([#86](#86)) ([5417064](5417064))
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 3.2.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant