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

refactor: replace null value with empty string #64

Merged
merged 1 commit into from
Feb 5, 2020

Conversation

frank-zsy
Copy link
Contributor

Signed-off-by: Frankzhaopku syzhao1988@126.com

Purpose | 本PR解决的问题

Changes | 本PR的更改

  • 将 null 值改为空字符串

Checklist | 检查

  • Are tests written for added code? | 新增代码是否添加测试用例
  • Yes
  • No because:
  • Did you introduce any new dependencies? | 如果添加新的依赖,请列出依赖并说明添加该依赖的原因
  • No
  • Yes
    • Dependency:
    • Reason:

Signed-off-by: Frankzhaopku <syzhao1988@126.com>
@menbotics menbotics bot added the kind/enhancement Category issues or prs related to enhancement. label Feb 5, 2020
@codecov-io
Copy link

Codecov Report

Merging #64 into master will decrease coverage by 0.11%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #64      +/-   ##
==========================================
- Coverage   26.19%   26.07%   -0.12%     
==========================================
  Files          18       18              
  Lines         439      441       +2     
  Branches       24       25       +1     
==========================================
  Hits          115      115              
- Misses        324      326       +2
Impacted Files Coverage Δ
app/service/data_format.ts 30.55% <0%> (-0.58%) ⬇️

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 489cf59...7118895. Read the comment docs.

@frank-zsy frank-zsy merged commit 07415d1 into wuhan2020:master Feb 5, 2020
@frank-zsy frank-zsy deleted the remove-null branch February 5, 2020 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Category issues or prs related to enhancement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants