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.

Sign up for GitHub

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

Jump to bottom

names(; imported=false) returns package imports #26267

Closed
mortenpi opened this issue Feb 28, 2018 · 0 comments
Closed

names(; imported=false) returns package imports #26267

mortenpi opened this issue Feb 28, 2018 · 0 comments
Assignees
@JeffBezanson
Labels
bug Indicates an unexpected problem or unintended behavior modules regression Regression in behavior compared to a previous version

Comments

@mortenpi
Copy link
Contributor

On Julia 0.7.0-DEV.4012:

module Foo
import REPL
foo(x) = x
end

julia> names(Foo; all=true, imported=false)
8-element Array{Symbol,1}:
 Symbol("#eval")   
 Symbol("#foo")    
 Symbol("#include")
 :Foo              
 :REPL             
 ...

Unless I am missing something, REPL should not be there in the returned array. On 0.6 it works the way I would expect -- imported packages do not get returned when you call names(Foo, true, false).

Ref: JuliaDocs/Documenter.jl#651
@StefanKarpinski StefanKarpinski added bug Indicates an unexpected problem or unintended behavior regression Regression in behavior compared to a previous version labels Feb 28, 2018
@JeffBezanson JeffBezanson self-assigned this Mar 2, 2018
JeffBezanson added a commit that referenced this issue Mar 2, 2018
@JeffBezanson
fix #26267, regression in names with imported=false
e10b1f4
mbauman added a commit that referenced this issue Mar 5, 2018
@mbauman
Merge remote-tracking branch 'origin/master' into mb/deprecatemultiva…
95a78df 
…luenonscalarindexedassignment

* origin/master: (28 commits)
  fix an optimizer bug in `invoke` with non-constant functions (#26301)
  lower top-level statements in such a way that the front-end knows (#26304)
  Make sure Sockets page has h1 header (#26315)
  fix doctests, and make them less prone to errors (#26275)
  FIx intro to manual chapter on types (#26312)
  Add a missing "that" (#26313)
  fix docstring for code_llvm (#26266)
  Remove the examples/ folder (#26153)
  download cert.pem from deps-getall, fixes #24903 (#25344)
  Slight update to doc string for at-enum to refer to instances (#26208)
  performance tweak in reverse(::String) (#26300)
  remove references to `TCPSocket` in Base docstrings (#26298)
  Deprecate adding integers to CartesianIndex (#26284)
  Deprecate conj(::Any), add real(::Missing) and imag(::Missing) (#26288)
  fix #26267, regression in `names` with `imported=false` (#26293)
  fix #25857, `typeinfo` problem in showing arrays with abstract tuple types (#26289)
  Add adjoint(::Missing) method (#25502)
  Use lowered form in logging macro (#26291)
  deprecate bin, oct, dec, hex, and base in favor of `string` and keyword args (#25804)
  deprecate `spawn(cmd)` to `run(cmd, wait=false)` (#26130)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JeffBezanson JeffBezanson

Labels
bug Indicates an unexpected problem or unintended behavior modules regression Regression in behavior compared to a previous version
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mortenpi @StefanKarpinski @JeffBezanson