Skip to content

test(cli): verify --cwd redirects config resolution #997#4796

Merged
escapedcat merged 1 commit into
masterfrom
test/cli-cwd-config-resolution
May 29, 2026
Merged

test(cli): verify --cwd redirects config resolution #997#4796
escapedcat merged 1 commit into
masterfrom
test/cli-cwd-config-resolution

Conversation

@escapedcat

@escapedcat escapedcat commented May 29, 2026

Copy link
Copy Markdown
Member

Fixes: #997

Adds a CLI test asserting that --cwd changes where config is resolved from, not just that the directory is valid: the same message is rejected by the process directory's config but accepted once --cwd points at a directory whose config allows it.

Adds a CLI test asserting that --cwd changes where config is resolved
from, not just that the directory is valid: the same message is rejected
by the process directory's config but accepted once --cwd points at a
directory whose config allows it.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@escapedcat escapedcat merged commit 51a2d7f into master May 29, 2026
14 checks passed
@escapedcat escapedcat deleted the test/cli-cwd-config-resolution branch May 29, 2026 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

[cli] missing tests for cwd argument in cli

1 participant