Change multiple_sep from : to ;

[rcannood]

Describe your changes

• Change default multiple_sep from : to ; to avoid conflicts with paths like s3://foo/bar.

Related issue(s)

Closes #xxxx

Type of Change

• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• New functionality (non-breaking change which adds functionality)
• Major change (non-breaking change which modifies existing functionality)
• Minor change (non-breaking change which does not modify existing functionality)
• Bug fix (non-breaking change which fixes an issue)
• This change requires a documentation update

Checklist

Requirements:

• I have read the CONTRIBUTING doc.
• I have performed a self-review of my code by checking the "Changed Files" tab.
• My code follows the code style of this project.

Tests:

• I have added tests that prove my fix is effective or that my feature works.
• New and existing unit tests pass locally with my changes.

Documentation:

• Proposed changes are described in the CHANGELOG.md.
• I have updated the documentation accordingly.

Test Environment

[Grifs]

Would it make sense to define a const somewhere and refer to that value in e.g. the nextflow script methods?

Additionally, we can perhaps set the @default(";") annotation in the abstract class Argument and let the arguments inherit the setting. TBC that it actually works.

[rcannood]

Would it make sense to define a const somewhere and refer to that value in e.g. the nextflow script methods?

Additionally, we can perhaps set the @default(";") annotation in the abstract class Argument and let the arguments inherit the setting. TBC that it actually works.

Good point!

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9c7a676) 92.48% compared to head (5cf8810) 92.48%.
Report is 3 commits behind head on develop.

❗ Current head 5cf8810 differs from pull request most recent head 79ee78b. Consider uploading reports for the commit 79ee78b to get more accurate results

Additional details and impacted files

@@           Coverage Diff            @@
##           develop     #645   +/-   ##
========================================
  Coverage    92.48%   92.48%           
========================================
  Files          129      129           
  Lines         3883     3883           
  Branches       644      644           
========================================
  Hits          3591     3591           
  Misses         292      292           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Grifs]

This PR looks fine to me now