Skip to content

feat: shell completion and Linux/macOS/Windows CI matrix - #3

Open
RogerCecilpot wants to merge 1 commit into
feat/246-output-errors-exit-codesfrom
feat/257-completion-windows-ci
Open

RogerCecilpot wants to merge 1 commit into
feat/246-output-errors-exit-codesfrom
feat/257-completion-windows-ci

Conversation

@RogerCecilpot

Copy link
Copy Markdown

Closes devlyft/devlyft#257.

Stacked on #2 (scaffold); retarget to main once that merges.

What

  • devlyft completion bash|zsh|fish|powershell replaces Cobra's default command, with install instructions for each shell and strict argument validation.
  • version: the line's shape is pinned by a test, and extra arguments are rejected.
  • CI: the test job is a matrix over ubuntu-latest, macos-latest and windows-latest (fail-fast: false). Each runs go test -race, builds the binary, runs version and generates every completion script. Windows uses Git Bash and calls go directly because make isn't available.

Notes

  • Required checks: the single checks / test becomes checks / test (ubuntu-latest), checks / test (macos-latest) and checks / test (windows-latest). The ruleset needs to list all three when the required checks are added.
  • Windows ANSI handling, Ctrl-C (exit 130) and the credential store come with #246 and #245. This matrix will run their tests on Windows.

🤖 Generated with Claude Code

@RogerCecilpot
RogerCecilpot force-pushed the feat/257-completion-windows-ci branch from a528af3 to 86e0e9d Compare October 25, 2026 12:56
@RogerCecilpot
RogerCecilpot changed the base branch from feat/235-scaffold to feat/246-output-errors-exit-codes October 25, 2026 12:57
Add an explicit `completion bash|zsh|fish|powershell` command with install
instructions per shell, replacing Cobra's default. Pin the shape of the
`version` line with a test.

The test job now runs on ubuntu, macOS and Windows, building the binary
and generating every completion script on each.

Refs devlyft/devlyft#257

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@RogerCecilpot
RogerCecilpot force-pushed the feat/246-output-errors-exit-codes branch from ab68d4d to dd9fe8c Compare October 25, 2026 13:00
@RogerCecilpot
RogerCecilpot force-pushed the feat/257-completion-windows-ci branch from 86e0e9d to d5618b3 Compare October 25, 2026 13:00
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.

1 participant