Skip to content

tests: add cli test for init with target path #1773

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

LorenzBischof
Copy link
Contributor

Tests for the issue described in #1769
Fixed by #1771

I had to remove the first test, because an interactive confirmation was added. The tests still worked in CI, because the command fails to allocate an interactive terminal.

@domenkozar
Copy link
Member

I'm not sure I understand why the CI fails

@LorenzBischof LorenzBischof force-pushed the push-uuuywwxlrtxx branch 2 times, most recently from 8cea369 to 5ae0f1a Compare March 18, 2025 10:23
@LorenzBischof
Copy link
Contributor Author

Should be fixed now. It seems my editor formatted something differently than the Git hook: https://github.com/cachix/devenv/compare/8cea369eb3b187954e441b9e044ca9e6b8f4daae..5ae0f1a783d25b7b824b69d9be0f91160d988a15

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