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

modified basic usage sample in readme #32

Merged
merged 1 commit into from
Nov 23, 2017
Merged

Conversation

SYctonu
Copy link

@SYctonu SYctonu commented Nov 23, 2017

#Hi,
I installed TemPy on my Windows machine and used IDLE (Python 3.6) to run the first sample code in the README, under basic usage. The HTML tags did not all render as given in the sample, so I made a few changes, mostly under # add tags and content later, tested them successfully on IDLE, and updated the README accordingly. This is my first OSS contribution, for a school assignment; you can find more details of my process here. I hope it helps!
Happy Thanksgiving

@coveralls
Copy link

Coverage Status

Coverage remained the same at 98.754% when pulling 1949def on SYctonu:readme-edit into b78e707 on Hrabal:master.

3 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 98.754% when pulling 1949def on SYctonu:readme-edit into b78e707 on Hrabal:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 98.754% when pulling 1949def on SYctonu:readme-edit into b78e707 on Hrabal:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 98.754% when pulling 1949def on SYctonu:readme-edit into b78e707 on Hrabal:master.

@Hrabal
Copy link
Owner

Hrabal commented Nov 23, 2017

Hi @SYctonu , thanks for your contribution! I'll add your changes also in the gh-pages branch, where I'm trying to write a complete documentation for the project.

@Hrabal Hrabal merged commit eb1dbc5 into Hrabal:master Nov 23, 2017
@SYctonu SYctonu deleted the readme-edit branch August 14, 2018 05:39
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.

3 participants