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

fix: config.Duration support int for yaml #149

Merged
merged 2 commits into from
Jul 13, 2021
Merged

fix: config.Duration support int for yaml #149

merged 2 commits into from
Jul 13, 2021

Conversation

GGXXLL
Copy link
Contributor

@GGXXLL GGXXLL commented Jul 7, 2021

No description provided.

@codecov
Copy link

codecov bot commented Jul 7, 2021

Codecov Report

Merging #149 (41aab37) into master (e6c6f98) will increase coverage by 0.05%.
The diff coverage is 60.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
+ Coverage   80.29%   80.34%   +0.05%     
==========================================
  Files          89       89              
  Lines        3724     3725       +1     
==========================================
+ Hits         2990     2993       +3     
+ Misses        534      533       -1     
+ Partials      200      199       -1     
Impacted Files Coverage Δ
config/config.go 83.18% <60.00%> (+0.15%) ⬆️
config/remote/remote.go 85.18% <0.00%> (+7.40%) ⬆️

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 e6c6f98...41aab37. Read the comment docs.

@GGXXLL GGXXLL closed this Jul 12, 2021
@GGXXLL GGXXLL deleted the config-duration branch July 12, 2021 07:18
@GGXXLL GGXXLL restored the config-duration branch July 13, 2021 01:39
@GGXXLL GGXXLL reopened this Jul 13, 2021
@Reasno Reasno merged commit 3ba6980 into DoNewsCode:master Jul 13, 2021
@GGXXLL GGXXLL deleted the config-duration branch July 13, 2021 01:48
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