Skip to content

Commit

Permalink
Change defaults for check_extras and check_weakdeps (#202)
Browse files Browse the repository at this point in the history
  • Loading branch information
lgoettgens authored Oct 24, 2023
1 parent 69a7655 commit be7ea3a
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 9 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Changed

- The minimum requirement for julia was raised from `1.0` to `1.4`. ([#221](https://github.com/JuliaTesting/Aqua.jl/pull/221))
- In `test_deps_compat`, the two subtests `check_extras` and `check_weakdeps` are now run by default. ([#202](https://github.com/JuliaTesting/Aqua.jl/pull/202)) [BREAKING]


## [0.7.4] - 2023-10-24
Expand Down
11 changes: 3 additions & 8 deletions src/deps_compat.jl
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,9 @@ each package listed under `deps`.
# Keyword Arguments
## Test choosers
- `check_extras = false`: If true, additionally check "extras". A NamedTuple
- `check_extras = true`: If true, additionally check "extras". A NamedTuple
can be used to pass keyword arguments with test options (see below).
- `check_weakdeps = false`: If true, additionally check "weakdeps". A NamedTuple
- `check_weakdeps = true`: If true, additionally check "weakdeps". A NamedTuple
can be used to pass keyword arguments with test options (see below).
## Test options
Expand All @@ -23,12 +23,7 @@ to the corresponding `check_\$x` keyword argument.
`@test` for "deps".
- `ignore::Vector{Symbol}`: names of dependent packages to be ignored.
"""
function test_deps_compat(
pkg::PkgId;
check_extras = false, # Maybe switch to `true` for next breaking version
check_weakdeps = false, # Maybe switch to `true` for next breaking version
kwargs...,
)
function test_deps_compat(pkg::PkgId; check_extras = true, check_weakdeps = true, kwargs...)
@testset "$pkg deps" begin
test_deps_compat(pkg, "deps"; kwargs...)
end
Expand Down
2 changes: 1 addition & 1 deletion test/test_smoke.jl
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ Aqua.test_all(
undefined_exports = false,
project_extras = false,
stale_deps = false,
deps_compat = (; check_extras = true, check_weakdeps = true),
deps_compat = false,
project_toml_formatting = false,
piracy = false,
)
Expand Down

0 comments on commit be7ea3a

Please sign in to comment.