gpedic / ex_q / 3e229942ba3e3bfe9fd9bc0d8fc43edd27811ec1

100%

push

github

Fix DSL scoping and params handling, update toolchain (1.2.0) (#8)

* Update toolchain to Elixir 1.20 and bump to 1.2.0

- Pin Elixir 1.20.4 / OTP 28.5 in .tool-versions
- Update credo (fixes compilation on Elixir >= 1.19), dialyxir, ex_doc and excoveralls
- Move preferred CLI envs to cli/0 and drop the duplicate source_url
- CI: test the Elixir floor (1.14) and latest (1.20), cache deps, use checkout v4

* Fix DSL scoping, params handling and docs

- queue/1 only rewrites top-level statements, so calls inside step
  functions (e.g. Kernel.inspect/1) are no longer turned into DSL calls
- queue/1 uses a hygienic variable instead of var!(queue), so a caller
  variable named queue is no longer shadowed
- DSL calls nested in if/case/other expressions and exec() before the
  last statement now raise at compile time instead of being dropped
- {params, opts} works for anonymous functions and run/6; malformed
  params and invalid :order values raise ArgumentError when added
- The duplicate name error no longer dumps the queue and its values
- Build the names MapSet at runtime in new/0 (dialyzer opaque warning)
- Correct typespecs, write a real moduledoc and make the doctests assert

* Fix README examples and document DSL limitations

* Require Elixir 1.15 and drop 1.14 from CI

cli/0 needs Elixir 1.15. Format and credo checks now run only on the
latest Elixir, since formatter output can differ between versions.

* Cover DSL inspect paths and drop unreachable clauses

Add tests for top-level inspect()/inspect(opts) in the DSL, a nested
Kernel.inspect/1, and empty blocks. Remove the dsl_statements(nil) and
non-call reject_nested_dsl! clauses, which the general clauses already
handle.

45 of 45 new or added lines in 1 file covered. (100.0%)

89 of 89 relevant lines covered (100.0%)

18.72 hits per line