search

moov-io / wire

FedWire funds service file parser and writer. The HTTP server is available in a Docker image and the Go package is available.
https://moov-io.github.io/wire/
Apache License 2.0
81 stars 38 forks source link

financialInstitution: combine duplicate validation functions #367

Closed atonks2 closed 7 months ago

atonks2 commented 7 months ago

What does this pull request do? This change combines duplicate Validate() funcs from the structs that include FinancialInstitution as a field onto the FinancialInstitution struct itself.

What Wire specification are you referencing? FAIM 3.0.4

codecov-commenter commented 7 months ago

Codecov Report

Merging #367 (5ed78d1) into master (e86b8f9) will decrease coverage by 5.96%. The diff coverage is 100.00%.

:exclamation: Current head 5ed78d1 differs from pull request most recent head 7ca8a0f. Consider uploading reports for the commit 7ca8a0f to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #367 +/- ## ========================================== - Coverage 74.86% 68.90% -5.96% ========================================== Files 73 79 +6 Lines 8682 9340 +658 ========================================== - Hits 6500 6436 -64 - Misses 1615 2337 +722 Partials 567 567 ```