Skip to content

DEV: use start for open-docs on Windows#869

Merged
lucascolley merged 2 commits into
data-apis:mainfrom
ColumbusLabs:codex/fix-windows-open-docs
Jul 25, 2026
Merged

DEV: use start for open-docs on Windows#869
lucascolley merged 2 commits into
data-apis:mainfrom
ColumbusLabs:codex/fix-windows-open-docs

Conversation

@ColumbusLabs

Copy link
Copy Markdown
Contributor

Closes #839.

Splits the open-docs task by platform:

  • Unix keeps the existing open build/index.html command.
  • Windows uses cmd /c start build/index.html.

The cmd /c wrapper is necessary because Pixi executes tasks with deno_task_shell, while start is a Windows cmd.exe built-in rather than a standalone executable.

Both variants retain the docs dependency, working directory, description, and default environment.

Validation:

  • Pixi 0.73 manifest parsing
  • Lock file unchanged and up to date
  • Full repository lint suite (dprint, Ruff, mypy, Pyright, Pyrefly, typos, actionlint, zizmor, and documentation checks)
  • Sphinx docs build with warnings treated as errors
  • Unix open-docs dry run renders the existing open build/index.html

Native Windows execution was not available locally; the Windows target-table selection and command form were verified against Pixi's target-task semantics and Windows cmd behavior.

@lucascolley
lucascolley self-requested a review July 25, 2026 09:23
@lucascolley lucascolley added the DX label Jul 25, 2026
Comment thread pixi.toml Outdated
Co-authored-by: Lucas Colley <lucas.colley8@gmail.com>
@lucascolley lucascolley changed the title DEV: use start for open-docs on Windows DEV: use start for open-docs on Windows Jul 25, 2026
@lucascolley lucascolley added this to the 0.11.1 milestone Jul 25, 2026
@lucascolley
lucascolley enabled auto-merge (squash) July 25, 2026 11:51
@lucascolley
lucascolley merged commit 3de63d4 into data-apis:main Jul 25, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DEV: open-docs should use start instead of open on windows

2 participants