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 quote parameter #16

Merged

Conversation

EwenBALOUIN
Copy link
Contributor

to assure retro compatibility of magic parser by using QUOTE_NONE in default

@EwenBALOUIN EwenBALOUIN added bug Something isn't working to review labels Nov 21, 2023
@EwenBALOUIN EwenBALOUIN self-assigned this Nov 21, 2023
}
)
rows, errors = schema.parse(b'column_1\n"test ""quoting""')
assert rows == [{"column_1": '"test ""quoting""'}]
Copy link
Contributor

Choose a reason for hiding this comment

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

Ah je pensais que ça ferait 'test quoting' !

Copy link
Contributor

Choose a reason for hiding this comment

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

ça marche toujours pour la partie portugal du coup? Ou tu veux test ça côté ext-is direct?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah bah non, ça ne te le fera jamais à ta place

Copy link
Contributor Author

Choose a reason for hiding this comment

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

pour la partie portugal, faut que je mette à jour mes tests côté ext mais ça casse rien

@EwenBALOUIN EwenBALOUIN merged commit 3ef8f7f into master Nov 21, 2023
@EwenBALOUIN EwenBALOUIN deleted the feature/int-280-handle-differents-csv-reader-type branch November 21, 2023 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants