Skip to content
Snippets Groups Projects

Make SASID an optional parameter

Merged Mick Veldhuis requested to merge make-sasid-optional into main
All threads resolved!

In the current implementation the SASID is a required workflow input. This makes no sense when the pipeline hasn't been called by a SAS process. Hence, this MR makes the SASID parameter optional; instead of using the SASID in name of the output MSs, the Observation ID is reused.

Merge request reports

Pipeline #100728 passed

Pipeline passed for 40b15b61 on make-sasid-optional

Approved by

Merged by Mick VeldhuisMick Veldhuis 1 month ago (Dec 10, 2024 1:55pm UTC)

Loading

Pipeline #101426 passed

Pipeline passed for 0964b072 on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Matthijs van der Wild
  • Matthijs van der Wild requested review from @wild

    requested review from @wild

  • Matthijs van der Wild approved this merge request

    approved this merge request

  • Mick Veldhuis resolved all threads

    resolved all threads

  • merged

  • Mick Veldhuis mentioned in commit 0964b072

    mentioned in commit 0964b072

  • Please register or sign in to reply
    Loading