search

essenciary / PkgSearch.jl

A Julia REPL utility for package discovery
MIT License
29 stars 2 forks source link

Remove all exports #2

Closed timholy closed 8 years ago

timholy commented 8 years ago

On julia-0.5 one was getting the following warning:

WARNING: using PkgSearch.next in module Main conflicts with an existing identifier.

That's because PkgSearch doesn't import Base: next. But I don't really see any reason to extend next, you just want your own private version of next, so the obvious thing to do is to not export it. Since in your README all your functions are scoped by the module, it seems reasonable to just delete all exports.

The first commit contains the only "real" change, the rest is just whitespace (I have emacs set to delete trailing whitespace, etc).

essenciary commented 8 years ago

Makes sense, I agree.

essenciary commented 8 years ago

Thanks for the contribution!