skdata icon indicating copy to clipboard operation
skdata copied to clipboard

Change readme file extensions to rst for nice rendering

Open danielunderwood opened this issue 9 years ago • 4 comments

danielunderwood avatar Apr 23 '16 16:04 danielunderwood

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?

jaberg avatar Apr 24 '16 20:04 jaberg

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.

danielunderwood avatar Apr 24 '16 20:04 danielunderwood

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

On Sun, Apr 24, 2016 at 4:54 PM, Daniel Underwood [email protected] 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 https://github.com/jaberg/skdata/pull/70#issuecomment-214034493

jaberg avatar Apr 24 '16 21:04 jaberg

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

danielunderwood avatar Apr 25 '16 00:04 danielunderwood