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

Small fix for problem of "line" instead of "lines" #5

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

Conversation

arruda
Copy link

@arruda arruda commented Dec 2, 2012

Just a small change, should fix #1

arruda and others added 3 commits December 2, 2012 21:15
small fix... now instead of line is lines
Fixing docstring to match the last commit change
@arruda
Copy link
Author

arruda commented Dec 3, 2012

Also added another thing that allows the user to do something like:

graph.add_lines([(1, 1), (2, 2), (3, 3)], label="func a",**{"fill":True})

@arruda arruda closed this Oct 17, 2014
@unbracketed
Copy link
Owner

@arruda Did you still want to merge a fix in?

@arruda
Copy link
Author

arruda commented Oct 30, 2014

Only if you want it, I closed it since it was open for some time.
I can reopen with you prefer =)

@unbracketed
Copy link
Owner

As you can probably tell, I'm not actively maintaining this repo. Happy to put in the change for posterity but I don't have plans for further development.

A belated thanks for your fix and feedback. Cheers.

@arruda
Copy link
Author

arruda commented Oct 31, 2014

No problem I'll reopen the PR.
Anyway thanks for the initiative of making this project.

@arruda arruda reopened this Oct 31, 2014
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.

Doco sample and reality differs
2 participants