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

Keep settlement date when parsing files. #960

Merged
merged 3 commits into from Jul 20, 2021

Conversation

eweise
Copy link
Contributor

@eweise eweise commented Jul 19, 2021

No description provided.

@eweise eweise requested a review from vxio as a code owner July 19, 2021 16:55
@codecov-commenter
Copy link

codecov-commenter commented Jul 19, 2021

Codecov Report

Merging #960 (1998e0f) into master (5ce71f8) will increase coverage by 0.89%.
The diff coverage is 100.00%.

❗ Current head 1998e0f differs from pull request most recent head 0879e4d. Consider uploading reports for the commit 0879e4d to get more accurate results

@@            Coverage Diff             @@
##           master     #960      +/-   ##
==========================================
+ Coverage   91.32%   92.22%   +0.89%     
==========================================
  Files          66       66              
  Lines        5247     6060     +813     
==========================================
+ Hits         4792     5589     +797     
- Misses        282      295      +13     
- Partials      173      176       +3     

adamdecaf
adamdecaf previously approved these changes Jul 19, 2021
vxio
vxio previously approved these changes Jul 19, 2021
Copy link
Member

@vxio vxio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two minor suggestions, but looks good!

validators.go Outdated Show resolved Hide resolved
validators.go Outdated Show resolved Hide resolved
@eweise eweise dismissed stale reviews from vxio and adamdecaf via e43fd92 July 20, 2021 01:19
eweise and others added 2 commits July 19, 2021 18:19
Co-authored-by: Vincent Xiao <vincentx72@gmail.com>
Co-authored-by: Vincent Xiao <vincentx72@gmail.com>
@vxio
Copy link
Member

vxio commented Jul 20, 2021

Appreciate the PR @eweise , I'm going to merge this in!

@vxio vxio merged commit a2ae6d6 into moov-io:master Jul 20, 2021
@eweise
Copy link
Contributor Author

eweise commented Jul 20, 2021

Awesome. Thank you both

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.

None yet

4 participants