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

Handle nil *time.Time Literal (PR #52) + test #73

Closed
wants to merge 2 commits into from

Conversation

quetz
Copy link

@quetz quetz commented Dec 15, 2018

This is basically code from PR #52 with added test case. Credits go to @RoarkeRandall

@codecov
Copy link

codecov bot commented Dec 15, 2018

Codecov Report

Merging #73 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
+ Coverage   80.72%   80.74%   +0.01%     
==========================================
  Files          17       17              
  Lines        2169     2171       +2     
==========================================
+ Hits         1751     1753       +2     
  Misses        310      310              
  Partials      108      108
Impacted Files Coverage Δ
dataset.go 95.54% <100%> (+0.05%) ⬆️

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 6803bc2...4949fa2. Read the comment docs.

@quetz
Copy link
Author

quetz commented Feb 6, 2019

Any issues on this PR that prevent it to be merged?

@doug-martin doug-martin mentioned this pull request Jun 8, 2019
@doug-martin
Copy link
Owner

I merged this into v6.1.0-rc with #82 I plan on getting a few other features in then publishing

@doug-martin doug-martin closed this Jun 8, 2019
@doug-martin doug-martin mentioned this pull request Jun 9, 2019
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