Skip to content

Aqua+explicitimports#31

Merged
Pangoraw merged 5 commits intoJuliaPluto:mainfrom
j-fu:aqua+explicitimports
Feb 16, 2026
Merged

Aqua+explicitimports#31
Pangoraw merged 5 commits intoJuliaPluto:mainfrom
j-fu:aqua+explicitimports

Conversation

@j-fu
Copy link
Copy Markdown
Contributor

@j-fu j-fu commented Feb 9, 2026

Add Aqua, ExplicitImports and UndocumentedNames tests.

@Pangoraw
Copy link
Copy Markdown
Member

tests are failing with:

  Expression: ExplicitImports.check_all_qualified_accesses_are_public(ThisPackage) === nothing
  NonPublicQualifiedAccessException
  Module `ExpressionExplorer` has explicit imports of names from modules in which they are not public (i.e. exported or declared public in Julia 1.11+):
  - `reverse` is not public in `Base.Iterators` but it was imported from `Base.Iterators` at `/home/runner/work/ExpressionExplorer.jl/ExpressionExplorer.jl/src/./explore.jl:450:42`

@fonsp
Copy link
Copy Markdown
Member

fonsp commented Feb 12, 2026

Weird, that function is public: https://docs.julialang.org/en/v1/base/iterators/#Base.Iterators.reverse

@Pangoraw
Copy link
Copy Markdown
Member

Probably only from 1.12 onwards:

julia> versioninfo()
Julia Version 1.11.9
Commit 53a02c0720c (2026-02-06 00:27 UTC)
Build Info:
  Official https://julialang.org/ release
Platform Info:
  OS: macOS (arm64-apple-darwin24.0.0)
  CPU: 10 × Apple M4
  WORD_SIZE: 64
  LLVM: libLLVM-16.0.6 (ORCJIT, apple-m1)
Threads: 1 default, 0 interactive, 1 GC (on 4 virtual cores)

julia> Base.ispublic(Base.Iterators, :reverse)
false

@fonsp
Copy link
Copy Markdown
Member

fonsp commented Feb 12, 2026

I guess that it's one of those functions that was public but they forgot to mark it

@Pangoraw
Copy link
Copy Markdown
Member

yes, let's disable for the LTS?

Copy link
Copy Markdown
Member

@Pangoraw Pangoraw left a comment

Choose a reason for hiding this comment

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

thanks

@testset "UndocumentedNames" begin
if isdefined(Docs, :undocumented_names) # >=1.11
@test isempty(filter!(e->e ≠ :try_compute_symbolreferences, Docs.undocumented_names(ThisPackage)))
end
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

it seems Aqua.test_all already does this?

Can you confirm

https://juliatesting.github.io/Aqua.jl/stable/test_all/#Aqua.test_all

@Pangoraw Pangoraw merged commit acce29f into JuliaPluto:main Feb 16, 2026
5 of 6 checks passed
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.

3 participants