Skip to content

Removed legacy warning about ',' separator in URIs#72

Merged
rdumusc merged 1 commit intoHBPVIS:masterfrom
rdumusc:master
May 19, 2017
Merged

Removed legacy warning about ',' separator in URIs#72
rdumusc merged 1 commit intoHBPVIS:masterfrom
rdumusc:master

Conversation

@rdumusc
Copy link
Copy Markdown

@rdumusc rdumusc commented May 19, 2017

FYI: I'm the 4th person to make a commit since the 1.5.0 release, to my disappointment nobody had updated the version so far. So I went through the history and came to the conclusion that at this point this is going to be a patch release, but please try not to forget to do it directly in the future...

Copy link
Copy Markdown

@hernando hernando left a comment

Choose a reason for hiding this comment

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

Weren't there any tests for this?

@rdumusc
Copy link
Copy Markdown
Author

rdumusc commented May 19, 2017

No, no tests, it was just printing a warning to users

@rdumusc rdumusc merged commit 3b62a36 into HBPVIS:master May 19, 2017
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.

4 participants