Thanks to visit codestin.com
Credit goes to github.com

Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: kjanat/runner
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v0.16.0
Choose a base ref
...
head repository: kjanat/runner
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: HEAD
Choose a head ref
  • 2 commits
  • 4 files changed
  • 1 contributor

Commits on Jul 1, 2026

  1. refactor(run): unify run alias dispatch; do not remove RunAliasCli (#72) 

    * refactor(run): route run alias dispatch through shared dispatch()

The `run` alias binary kept a second copy of resolver-override building
and command dispatch in `dispatch_run_alias`, parallel to `dispatch`.
Collapse that duplication: map a parsed `RunAliasCli` onto a typed `Cli`
(bare invocation -> `command: None` dashboard; otherwise `Command::Run`)
and funnel through the single `dispatch` entry point. Type-safe `Cli`
construction, not argv string rewriting.

Issue #52 proposed rewriting argv to ["runner", "run", ...] and
re-parsing through `Command::Run`. Not viable: `RunAliasCli` disables
clap's --help/--version so a trailing one forwards to the task (`run test
--help` runs the task with --help), whereas `Command::Run` inherits the
enabled global help/version and short-circuits to clap help. The flat
alias command also backs the `run` completion factory (task positional,
no subcommand leakage) and the `run` man page. So the struct stays;
only the dispatch duplication is removed.

Verification: cargo build/test (706 lib + integration)/clippy/dprint all
green under --all-features; manually confirmed alias semantics intact
(task dispatch, builtin-name-as-task, --help/-V forwarding, own
help/version, bare dashboard, shared override errors, chains) plus `run`
man page and completion generation.

Closes #52

* docs(changelog): correct run-alias 'no behavior change' claim

bare `run -k`/`-K` (no task, no -s/-p) now maps to command: None and
drops the inert chain-failure flag before override building, so an
opposite-polarity RUNNER_KILL_ON_FAIL/RUNNER_KEEP_GOING or [chain] config
no longer triggers resolve_failure_policy's cross-source conflict. Reword
CHANGELOG + dispatch_run_alias doc to state the delta; add regression test
pinning the bare-path flag drop.

* docs(changelog): move run alias note to unreleased

* fix(lsp): borrow command in early dispatch check
    @kjanat
    kjanat authored Jul 1, 2026
    Configuration menu
    Copy the full SHA
    21d6991 View commit details
    Browse the repository at this point in the history

  2. feat(config): discover .runner.toml and .config/ variants 

    `load` searched only `dir/runner.toml`. Now walk CONFIG_DIRS × plain/
dotted CONFIG_FILENAME in precedence order — root beats `.config/`,
canonical beats dotfile — so hidden and XDG-style placements are found.
First existing candidate wins; non-NotFound read errors still propagate.

LSP `is_runner_toml` accepts the dotfile form too, keeping diagnostics
active on `.runner.toml`. Covered by four discovery/precedence tests.
    @kjanat
    kjanat committed Jul 1, 2026
    Configuration menu
    Copy the full SHA
    ff9ce62 View commit details
    Browse the repository at this point in the history
Loading