Center-for-Research-Libraries / crl-serials-validator

Validate bibliographic and holdings data for shared print.
GNU General Public License v3.0
0 stars 1 forks source link

Remove holdings_have_no_years error category. #28

Closed nflorin closed 2 years ago

nflorin commented 2 years ago

The category holdings_have_no_years is deceiving for a number of reasons -- it calls out valid holdings without years in them, holdings that are entirely supplements or indexes (from neither of which do we gather years), and holdings with years expressed in a non-Gregorian calendar, and holdings with years that are simply difficult to parse due to some formatting issue.

Instead of an error, we should add it to the header of an output column. This way we'll preserve the data when it's useful, without suggesting that it's inherently a problem.

nflorin commented 2 years ago

Done.