Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check for spaces in IDs in Sample Sheet #67

Open
ThHarbig opened this issue Oct 14, 2021 · 1 comment
Open

Check for spaces in IDs in Sample Sheet #67

ThHarbig opened this issue Oct 14, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@ThHarbig
Copy link

ThHarbig commented Oct 14, 2021

I got a "too many arguments" error for one of the commands when I had sample IDs with spaces in the sample sheet. I don't recall which command it was exactly but this should be an issue for any command. I'd suggest to either throw an error when the sample sheet is incorrect in this regard or to automatically get rid of spaces in the IDs.

Thanks!

@ThHarbig ThHarbig added the bug Something isn't working label Oct 14, 2021
@d4straub
Copy link
Collaborator

That could be considered, yes. Thanks for the suggestion!
I'd not call it a "bug" though, because it came from user-error not pipeline error. Essentially it is adding a check and reporting a more informative error message.

@d4straub d4straub added enhancement New feature or request and removed bug Something isn't working labels Oct 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants