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

cmdline options changed #179

Merged

Conversation

xjasg
Copy link

@xjasg xjasg commented Apr 7, 2024

In order to maintain the same syntax for the command line parameters, the required entries for input / output file or directory have been uniformized.

@caetera caetera added the duplicate This issue or pull request already exists label Apr 8, 2024
@caetera
Copy link
Collaborator

caetera commented Apr 8, 2024

This seems to be a duplicate of #178.

@xjasg
Copy link
Author

xjasg commented Apr 8, 2024

@caetera That's partly true, but #178 contains commits that are off-topic. That's why I created this PR. Wolfgang would have to take back 178. Unfortunately I have no rights to do so.

@caetera
Copy link
Collaborator

caetera commented Apr 9, 2024

@xjasg thanks for the explanation. I will merge this PR into next release branch.

@xjasg
Copy link
Author

xjasg commented Apr 9, 2024

@caetera Okay, thank you.

@caetera caetera mentioned this pull request Apr 18, 2024
@ypriverol ypriverol merged commit 71e0340 into compomics:master May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants