Skip to content

feat: add --skip-local-tests flag to deploy command#112

Merged
dudutwizer merged 1 commit into
mainfrom
dt-20251105-skip-local-tests
Nov 6, 2025
Merged

feat: add --skip-local-tests flag to deploy command#112
dudutwizer merged 1 commit into
mainfrom
dt-20251105-skip-local-tests

Conversation

@dudutwizer

Copy link
Copy Markdown
Contributor

Summary

  • Added --skip-local-tests flag to xpander deploy command to allow skipping local Docker container tests
  • Users can now proceed directly to deployment without waiting for local validation
  • Maintains backward compatibility with default behavior (tests still run by default)

Changes

Usage

# Deploy with local tests (default behavior)
xpander deploy

# Deploy and skip local Docker tests
xpander deploy --skip-local-tests

Test plan

  • Build passes successfully
  • All unit tests pass (24/24)
  • ESLint validation passes
  • Help text displays new flag correctly
  • Code follows project conventions (2-space indent, single quotes, async/await)

🤖 Generated with Claude Code

Add a new --skip-local-tests flag to the xpander deploy command that allows users to skip local Docker container testing and proceed directly to deployment.

Changes:
- Add --skip-local-tests option to deploy command (src/commands/deploy.ts)
- Update deployAgent function to accept skipLocalTests parameter (src/commands/agent/interactive/deploy.ts)
- Modify buildAndSaveDockerImage to conditionally skip tests when flag is set (src/utils/custom_agents_utils/docker.ts)

Usage:
  xpander deploy --skip-local-tests

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@dudutwizer
dudutwizer force-pushed the dt-20251105-skip-local-tests branch from 0a2794a to 860cf4e Compare November 6, 2025 01:04
@dudutwizer
dudutwizer merged commit ad43c57 into main Nov 6, 2025
3 checks passed
dudutwizer pushed a commit that referenced this pull request Nov 6, 2025
The previous PR #112 added the flag to the top-level deploy command but missed the agent subcommand path. This adds the missing flag to `xpander agent deploy`.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
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.

2 participants