Skip to content

Conversation

@jeffreyaven
Copy link
Member

This PR implements tab completion functionality for the stackql-deploy CLI as requested in issue #65.

Changes

  • Added new completion command to generate shell completion scripts
  • Support for bash, zsh, fish, and PowerShell shells
  • Leverages Click's built-in shell completion functionality
  • Updated README with detailed setup instructions
  • Enables tab completion for all subcommands and flags

Usage

Users can enable tab completion by:

  1. Running stackql-deploy completion <shell> to get setup instructions
  2. Adding the appropriate line to their shell config file
  3. Restarting their terminal or sourcing the config

Resolves #65

🤖 Generated with Claude Code

- Add new 'completion' command to generate shell completion scripts
- Support bash, zsh, fish, and PowerShell shells
- Leverage Click's built-in shell completion functionality
- Update README with detailed setup instructions for each shell
- Enable tab completion for subcommands and flags across all shells

Resolves #65

Co-authored-by: Jeffrey Aven <jeffreyaven@users.noreply.github.com>
@jeffreyaven
Copy link
Member Author

@claude can you fix https://github.com/stackql/stackql-deploy/actions/runs/18422097076, ruff linting error

@claude
Copy link
Contributor

claude bot commented Oct 11, 2025

Claude finished @jeffreyaven's task —— View job


✅ Fixed ruff linting error

  • Read the changed files to understand the implementation
  • Check for repo-specific instructions in CLAUDE.md
  • Identify the specific ruff linting error
  • Fix the linting error (line length issue in completion command)
  • Commit and push the changes

Fixed: Line 389 in stackql_deploy/cli.py exceeded the 120 character limit configured in ruff settings. I broke the long @click.argument() decorator into multiple lines for better readability and compliance with the linting rules.

The fix has been committed and pushed to the PR branch. The GitHub Actions workflow should now pass the ruff linting checks.


claude bot and others added 2 commits October 11, 2025 01:09
…rements

Co-authored-by: Jeffrey Aven <jeffreyaven@users.noreply.github.com>
@jeffreyaven jeffreyaven merged commit 6c11a85 into main Oct 11, 2025
3 checks passed
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.

[FEATURE] Add tab completion for stackql-deploy CLI commands

2 participants