Skip to content
This repository has been archived by the owner on Feb 4, 2020. It is now read-only.

#109 permissive marcreader #144

Merged
merged 7 commits into from
Dec 5, 2019
Merged

Conversation

petrus-v
Copy link
Contributor

@petrus-v petrus-v commented Dec 3, 2019

This is a proposal to address #109 which allow to read files that may contains record with some structural errors.

Pierre Verkest added 3 commits December 3, 2019 21:39
as MARCReaderStringTest was MARCReaderFileTest child some test dedicated to
MARCReaderFileTest was running in MARCReaderStringTest test class with no additionnal values
as they were not using self.reader
This allow to read a large amont of data that may contains
wrong structural records which will be ignore to give a chance
to read as much of possible of recods in a file even one is
incorect
@Wooble
Copy link
Collaborator

Wooble commented Dec 3, 2019

FWIW the 2.6 failure is kind of obnoxious and I'm +1 on dropping support at this point. (I'd also be for dropping py2.7 support but that may be a harder sell...)

@petrus-v
Copy link
Contributor Author

petrus-v commented Dec 3, 2019

@Wooble thanks for your quick feed back I'll remove test against 2.6 them !

@edsu
Copy link
Owner

edsu commented Dec 5, 2019

This looks awesome @petrus-v & @Wooble -- thanks for both your efforts!

@edsu edsu merged commit 180f8da into edsu:master Dec 5, 2019
@Wooble Wooble mentioned this pull request Dec 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants