Skip to content

change output folder based on run type#57

Merged
harshavemula-ua merged 3 commits intomainfrom
routing-script-merge
Feb 19, 2026
Merged

change output folder based on run type#57
harshavemula-ua merged 3 commits intomainfrom
routing-script-merge

Conversation

@quinnylee
Copy link
Collaborator

@quinnylee quinnylee commented Feb 11, 2026

See CIROH-UA/datastreamcli#51

This closes issue #58

Additions

Removals

Changes

Testing

Screenshots

Notes

Todos

Checklist

  • PR has an informative and human-readable title
  • Changes are limited to a single goal (no scope creep)
  • Code can be automatically merged (no conflicts)
  • Code follows project standards (link if applicable)
  • Passes all existing automated tests
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future todos are captured in comments
  • Visually tested in supported browsers and devices (see checklist below 👇)
  • Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
  • Reviewers requested with the Reviewers tool ➡️

Testing checklist

Target Environment support

  • Windows
  • Linux
  • Browser

Accessibility

  • Keyboard friendly
  • Screen reader friendly

Other

  • Is useable without CSS
  • Is useable without JS
  • Flexible from small to large screens
  • No linting errors or warnings
  • JavaScript tests are passing

@quinnylee
Copy link
Collaborator Author

CI failures seem to be related to #55

@quinnylee quinnylee marked this pull request as ready for review February 11, 2026 18:08
@quinnylee
Copy link
Collaborator Author

Marking this as ready for review despite the ci failures because it passed all the channel routing tests locally (except the obvious one, the s3 one due to not having the right credentials)

@harshavemula-ua harshavemula-ua merged commit a875b9e into main Feb 19, 2026
17 of 59 checks passed
@quinnylee quinnylee deleted the routing-script-merge branch February 24, 2026 16:44
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.

3 participants