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

Change readme file extensions to rst for nice rendering #70

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

Conversation

danielunderwood
Copy link

No description provided.

@jaberg
Copy link
Owner

jaberg commented Apr 24, 2016

Thanks for the PR! I think there's a tension between the rendering on github and the rendering on pypi. Would you mind looking up best practice on dealing with that with Python packages?

@danielunderwood
Copy link
Author

I wasn't aware of the pypi issue. It looks like the best practice is using reStructuredText since both GitHub and pypi support rendering of rst. It looks like the readme files for this project follow the syntax of rst, so no change is needed beyond an extension change.

I'll go ahead and make that change with the PR if that seems like an acceptable solution.

@jaberg
Copy link
Owner

jaberg commented Apr 24, 2016

Awesome, thanks for the info and that change would be great.

On Sun, Apr 24, 2016 at 4:54 PM, Daniel Underwood notifications@github.com
wrote:

I wasn't aware of the pypi issue. It looks like the best practice is using
reStructuredText since both GitHub and pypi support rendering of rst. It
looks like the readme files for this project follow the syntax of rst, so
no change is needed beyond an extension change.

I'll go ahead and make that change with the PR if that seems like an
acceptable solution.


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#70 (comment)

@danielunderwood danielunderwood changed the title Change readme file extensions to md for markdown rendering Change readme file extensions to rst for nice rendering Apr 24, 2016
@danielunderwood
Copy link
Author

I have pushed commits to change the readme files to the rst extension and highlight code in blocks in the main readme.

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.

None yet

2 participants