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.

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

feat: more log for bodyParser #3809

Merged
merged 1 commit into from
Jul 17, 2019
Merged

feat: more log for bodyParser #3809

merged 1 commit into from
Jul 17, 2019

Conversation

atian25
Copy link
Member

@atian25 atian25 commented Jul 10, 2019

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

@atian25 atian25 requested a review from fengmk2 July 10, 2019 03:43
@fengmk2
Copy link
Member

fengmk2 commented Jul 15, 2019

@atian25 算 feature,不是 bugfix。

@atian25 atian25 changed the title fix: more log for bodyParser feat: more log for bodyParser Jul 15, 2019
@atian25 atian25 force-pushed the bodyParser-error-msg branch from 4366fa1 to a61adc5 Compare July 17, 2019 06:17
@codecov
Copy link

codecov bot commented Jul 17, 2019

Codecov Report

Merging #3809 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #3809   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files          34      34           
  Lines         943     945    +2     
======================================
+ Hits          943     945    +2
Impacted Files Coverage Δ
config/config.default.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

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

@atian25 atian25 merged commit 29a2f2f into master Jul 17, 2019
@atian25 atian25 deleted the bodyParser-error-msg branch July 17, 2019 07:09
popomore pushed a commit that referenced this pull request Jul 17, 2019
fix: more log for bodyParser (#3809)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants