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

Add file encoding detection #100

Open
wants to merge 153 commits into
base: master
Choose a base branch
from

Conversation

exaucae
Copy link

@exaucae exaucae commented Oct 14, 2021

Inspired by this blogpost idea .

The encoding should now be inferred if not specified in the config properties.

Closes #82

@kstrub
Copy link

kstrub commented Feb 10, 2022

Any updates on this? We would need this feature for our application. The merge would be highly appreciated!
✌ Cheers

@Bunlong
Copy link
Owner

Bunlong commented Feb 12, 2022

@kstrub PR will be merged in next version 4.1.x. Thank you so much!

@Bunlong
Copy link
Owner

Bunlong commented Feb 14, 2022

@exaucae Could you please resolve the conflicts?

@exaucae
Copy link
Author

exaucae commented Feb 14, 2022

Sure, I'll ping you when done, @Bunlong, tonight.

@exaucae
Copy link
Author

exaucae commented Feb 25, 2022

If you test it and are okay with changes, we can move on, @Bunlong .

@antoinerousseau
Copy link

Why is this awesome PR not merged?

@Ehsan-Yari
Copy link

any update on this ??

@mdidehbanmehr
Copy link

Is this PR expected to be merged?

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.

How to detect the encoding of the loaded file before displaying data ?
8 participants