Skip to content

61 copy input files v2#62

Merged
rogerwq merged 2 commits intomainfrom
61-copy-input-files-v2
Jan 18, 2026
Merged

61 copy input files v2#62
rogerwq merged 2 commits intomainfrom
61-copy-input-files-v2

Conversation

@rogerwq
Copy link
Collaborator

@rogerwq rogerwq commented Jan 18, 2026

Breaking change to standardize input/output folder conventions for job execution.

Changes (v0.4.0):

  • BREAKING: Input files now use inputs/ subfolder convention
    • Dependency outputs are copied to {job}/inputs/ folder
    • Workflow input_files/ contents copied to first job's inputs/ folder
    • Scripts should read from ./inputs/ and write to ./outputs/
    • New Feature: Workflows can include an input_files/ folder at root
    • Contents automatically copied to first job's inputs/ before execution

Test Plan

  • Create workflow with input_files/ folder and verify contents copied to first job's inputs/
  • Run multi-job workflow and verify dependency outputs copied to inputs/ subfolder
  • Verify scripts can read from ./inputs/ and write to ./outputs/
  • Test backward compatibility with existing workflows (may need migration)

  When running a workflow in headless mode, silva now checks for an
  'input_files/' folder at the workflow root. If found, all contents
  are copied to the first job folder before execution. If not found,
  a hint is printed to inform the user.
  BREAKING CHANGE: Input files are now copied to inputs/ subfolder instead of job root.

  - Dependency outputs copied to {job}/inputs/ folder
  - Workflow input_files/ contents copied to first job's inputs/ folder
  - Scripts should read from ./inputs/ and write to ./outputs/

  This provides clear separation between inputs and outputs
@rogerwq rogerwq merged commit cf8c9db into main Jan 18, 2026
0 of 4 checks passed
@rogerwq rogerwq deleted the 61-copy-input-files-v2 branch January 18, 2026 17:02
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