Skip to content

Conversation

@jgrund
Copy link
Member

@jgrund jgrund commented Aug 21, 2025

create a recording because it looks for the recordings under
~/.cargo.../testcmds/recording-name.json. Instead, we should use the
runtime CARGO_MANIFEST_DIR, which will point to the crate where
commandeer is being imported.

  create a recording because it looks for the recordings under
  ~/.cargo.../testcmds/recording-name.json. Instead, we should use the
  runtime CARGO_MANIFEST_DIR, which will point to the crate where
  commandeer is being imported.

Signed-off-by: William Johnson <[email protected]>
if !file_path.exists() {
tokio::fs::create_dir_all(
file_path.parent().unwrap_or_else(|| {
panic!("Couldn't get parent of recording {}", file_path.display())
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Return anyhow error here

Signed-off-by: William Johnson <[email protected]>
@jgrund jgrund added the bug Something isn't working label Aug 21, 2025
@jgrund jgrund merged commit 9d6c087 into master Aug 21, 2025
1 check passed
@jgrund jgrund deleted the johnsonw/fix-directory-mapping branch August 21, 2025 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants