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

fix: Fix parse record error verbiage #177

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JarvisVon
Copy link

@JarvisVon JarvisVon commented Jun 24, 2024

Fix issue where, with 9 columns, an error is returned that states "The line contains an invalid number of fields. Was 9 but expected 9"

Move to use self.expected_fields in the raised exception, to reduce code redundancy and use the value stored in the object itself

Closes #175

@JarvisVon JarvisVon changed the title Fix parse record error verbiage fix: Fix parse record error verbiage Jun 24, 2024
@JarvisVon JarvisVon force-pushed the feature/fix-was-9-but-expected-9-error branch from a642d70 to 5f87a81 Compare June 24, 2024 22:55
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.

Fix parse record error verbiage
1 participant