Skip to content

Conversation

@BibMartin
Copy link
Contributor

addresses #281

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a link to Leaflet Data Vis Framework with the phrase:

For more information see URL

@ocefpaf
Copy link
Member

ocefpaf commented Dec 4, 2015

👍

@ocefpaf ocefpaf self-assigned this Dec 4, 2015
@ocefpaf ocefpaf added the documentation Documentation about a certain topic should be added label Dec 4, 2015
@ocefpaf ocefpaf added this to the v0.2.0 milestone Dec 4, 2015
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was thinking that this should be at the bottom. That is consistent with the other docstrings I am writing and makes sense. You read what is here first, then you follow the URL for more info 😉

@BibMartin
Copy link
Contributor Author

I was thinking that this should be at the bottom. That is consistent with the other docstrings I am writing and makes sense. You read what is here first, then you follow the URL for more info 😉

Done and squashed.

string it not working yet, right?

That's true. BUt it's coming from #265 in a few days (I promise).

ocefpaf added a commit that referenced this pull request Dec 4, 2015
RegularPolygonMarker docstring
@ocefpaf ocefpaf merged commit f84de18 into python-visualization:master Dec 4, 2015
@ocefpaf
Copy link
Member

ocefpaf commented Dec 4, 2015

That's true. But it's coming from #265 in a few days (I promise).

No problem. I'd rather have that added in all docstrings now than implementing it later and forgetting about the doscstrings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Documentation about a certain topic should be added

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants