Add option to specify UMI separator in extract 2

[janoppelt]

Add option to specify UMI separator in umi_tools extract. The default stays as it is now ("_"). Allows users to change it to other strings. 2

[IanSudbery]

Hi @TomSmithCGAT can we set up the CI to run on PRs as well as commits?

@opplatek This looks good, but I'll need to run our tests.

[TomSmithCGAT]

I just need to finish off #546 (add py 3.5) and then merge and it should run on all pushes

UMI-tools/.github/workflows/nosetests.yml

Line 3 in cd6cfa7

on: [push]

I've blocked out tomorrow for just UMI-tools so I'll do this first thing and then try and resolve the deterministic issues

[TomSmithCGAT]

Merged #546 and set to run CI on PRs and pushes. Need to work out how to get it to run on an existing PRs now though...
If all else fails, I guess it's a re-issue PR or add new commit with e.g empty line

[TomSmithCGAT]

💡 Just need to close and re-open and github treats it as a new PR 😁

[TomSmithCGAT]

@opplatek - Am I right in understanding that the two PRs from your patch-1 and patch-2 branches both need to be merged? I don't think the testing is appropriate for the separate PRs:

• patch-1 fails the tests because of an issue resolved in patch-2
• patch-2 passes the tests because it doesn't have any effect without the changes in patch-1

Could you please merge the two patches and issue a new PR 🙏 Thank you!

[janoppelt]

Hi @TomSmithCGAT . Sorry for the confusion. The new push of the changes is at #551.