Skip to content

Document --ignored and --nocapture flags in release workflow test - #269

Closed
codekiln with Copilot wants to merge 2 commits into
mainfrom
copilot/sub-pr-268
Closed

Document --ignored and --nocapture flags in release workflow test#269
codekiln with Copilot wants to merge 2 commits into
mainfrom
copilot/sub-pr-268

Conversation

Copilot AI commented Nov 24, 2025

Copy link
Copy Markdown
Contributor

Addressed review question about test flags in the pre-release validation step of the release workflow.

Context

The cargo test command at line 75 of .github/workflows/release.yml uses two flags:

  • --ignored: Runs tests marked with #[ignore] attribute (the deployment test is explicitly ignored for normal test runs)
  • --nocapture: Displays test output in real-time rather than capturing it, useful for monitoring long-running integration tests

No code changes required—this was a clarification question about existing functionality.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

codekiln and others added 2 commits November 24, 2025 10:46
Add comprehensive pre-release validation job that runs full lifecycle
deployment test before creating releases. This quality gate ensures
the complete deployment workflow (create, update, delete) works
correctly before publishing releases.

Changes:
- Add pre-release-validation job to release.yml
- Run test_deployment_workflow_full_lifecycle before release creation
- Configure 45-minute timeout (test takes ~20-30 minutes)
- Require LANGSMITH_API_KEY and LANGSMITH_WORKSPACE_ID secrets
- Block release if validation fails
- Update CI/CD documentation with new workflow step

The test validates:
- Creating fresh deployment with unique name
- Waiting for deployment to be READY
- Patching deployment (triggers new revision)
- Waiting for new revision to be DEPLOYED
- Deleting deployment (cleanup via DeploymentGuard)

Fixes #207

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

Co-Authored-By: Claude <noreply@anthropic.com>
Base automatically changed from codekiln/207-pre-release-validation to main November 24, 2025 15:50
Copilot AI changed the title [WIP] Update pre-release validation tests in release workflow Document --ignored and --nocapture flags in release workflow test Nov 24, 2025
Copilot AI requested a review from codekiln November 24, 2025 15:54
@codekiln codekiln closed this Nov 24, 2025
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