Skip to content

Validate output artifact file paths#42

Merged
stacknil merged 1 commit into
mainfrom
stacknil/validate-output-file-paths
May 18, 2026
Merged

Validate output artifact file paths#42
stacknil merged 1 commit into
mainfrom
stacknil/validate-output-file-paths

Conversation

@stacknil
Copy link
Copy Markdown
Owner

Summary

  • reject output artifact paths that already exist as directories before writing tables, JSON, or plots
  • reuse the output path guard across table/summary writes and plot rendering
  • cover run and plot CLI failures for directory-valued artifact paths without tracebacks

Tests

  • pytest tests/test_cli_errors.py tests/test_pipeline_e2e.py
  • pytest
  • python -m telemetry_window_demo.cli run --config configs/default.yaml

@stacknil stacknil merged commit 4a31666 into main May 18, 2026
2 checks passed
@stacknil stacknil deleted the stacknil/validate-output-file-paths branch May 18, 2026 03:25
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.

1 participant