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

Rename the command line functions calls #144

Closed
thomasyu888 opened this issue May 2, 2020 · 2 comments · Fixed by #188 or #190
Closed

Rename the command line functions calls #144

thomasyu888 opened this issue May 2, 2020 · 2 comments · Fixed by #188 or #190
Milestone

Comments

@thomasyu888
Copy link
Member

Calls like:

challengeutils setevaluationquota
challengeutils killdockeroverquota
challengeutils downloadsubmission

Should probably just be

challengeutils set-evaluation-quota
challengeutils kill-docker-over-quota
challengeutils download-submission

This will be a breaking change.

@thomasyu888 thomasyu888 changed the title Refactor the command line functions calls Rename the command line functions calls May 2, 2020
@thomasyu888 thomasyu888 added this to the 3.0 milestone May 9, 2020
@vpchung
Copy link
Member

vpchung commented May 9, 2020

While we're at it, we should standardize the parameter names and short flags as well, e.g.

  • mirrorwiki uses --forceupdate with no short flag vs. annotatesubmission uses --force with the short flag, -f

@thomasyu888
Copy link
Member Author

Good catch, I actually update --forceupdate to --force in #143 so might as well add the -f

@thomasyu888 thomasyu888 mentioned this issue Jun 17, 2020
3 tasks
@thomasyu888 thomasyu888 linked a pull request Jul 7, 2020 that will close this issue
3 tasks
@thomasyu888 thomasyu888 mentioned this issue Jul 7, 2020
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants