Skip to content

Conversation

@MattWellie
Copy link
Collaborator

Purpose

  • The temp-dir should be a String, so that we don't check for its existence
  • It's intentionally deleted by the next Stage, so a re-run of later stages of the workflow would accidentally re-trigger running the combiner
  • e.g. here

Proposed Changes

  • Make the temp-directory Path a String, so its existence isn't checked
  • Make the String into a Path object when it's used to generate temp output paths

@MattWellie MattWellie requested a review from EddieLF August 14, 2025 01:04
@MattWellie MattWellie mentioned this pull request Oct 17, 2025
3 tasks
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