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 utf-8 encoding when reading the readme.rst #47

Closed
wants to merge 2 commits into from
Closed

fix utf-8 encoding when reading the readme.rst #47

wants to merge 2 commits into from

Conversation

chanansh
Copy link

fixing this encoding resolves the installation issue #46 in Windows.

@coveralls
Copy link

coveralls commented Jan 31, 2017

Coverage Status

Coverage remained the same at 82.418% when pulling 54a1377 on chanansh:fix_setup.py_encoding_to_utf-8_for_windows into cf9d4f0 on theskumar:master.

@chanansh
Copy link
Author

Hi @theskumar . please help me pass the tests. I don't know why it fails.

@theskumar
Copy link
Owner

Looking at it.

@aniketmaithani
Copy link

@chanansh I think it's a simple flake8 issue. Just fix your 1 indent = 4 spaces :P in your editor settings and voila IT SHALL PASS :D

here is the error message :)

/home/travis/build/theskumar/python-dotenv/setup.py:6:1: W191 indentation contains tabs
/home/travis/build/theskumar/python-dotenv/setup.py:7:1: W191 indentation contains tabs
/home/travis/build/theskumar/python-dotenv/setup.py:9:1: W191 indentation contains tabs
/home/travis/build/theskumar/python-dotenv/setup.py:10:1: W191 indentation contains tabs
/home/travis/build/theskumar/python-dotenv/setup.py:11:1: W191 indentation contains tabs
/home/travis/build/theskumar/python-dotenv/setup.py:14:1: E101 indentation contains mixed spaces and tabs

theskumar pushed a commit that referenced this pull request Feb 3, 2017
@theskumar
Copy link
Owner

This is fixed in v0.6.3, thanks for PR @chanansh

@theskumar theskumar closed this Feb 3, 2017
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.

5 participants