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

No BOM character at the beginning of the csv #186

Closed
AuroreM opened this issue Jul 18, 2017 · 3 comments
Closed

No BOM character at the beginning of the csv #186

AuroreM opened this issue Jul 18, 2017 · 3 comments

Comments

@AuroreM
Copy link
Contributor

AuroreM commented Jul 18, 2017

I used this library to create csv then opened with Excel and I encountered some issue with special characters :
image

I found that I had to add the BOM character U+FEFF at the beginning of the CSV string.
I've got a branch ready for a pull request to add a --with-BOM option :

image

But I can not push my branch. Could you help me please ?
(It's my first contribution to an open source repo :))

@4ian
Copy link

4ian commented Jul 18, 2017

But I can not push my branch. Could you help me please ?
(It's my first contribution to an open source repo :))

The way to submit pull requests with GitHub is to Fork the repository:

image

You'll have a fork for your username (https://github.com/AuroreM/json2csv). Then git clone your fork and do the changes in it (or add https://github.com/AuroreM/json2csv as a remote to your existing git repository where you've made the changes. Then push to that remote).

Once you have your branch pushed in your fork you can open a Pull Request and GitHub will show your branch or your fork. :)

@AuroreM
Copy link
Contributor Author

AuroreM commented Jul 18, 2017

Thanks @4ian, the Pull Request is opened : #187 :)

@knownasilya
Copy link
Collaborator

Ah, missed this issue. Thanks for catching this.

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

No branches or pull requests

3 participants