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

fix(test): parse test command arguments #1686

Merged
merged 6 commits into from
May 25, 2024
Merged

Conversation

cmatsuoka
Copy link
Contributor

Let the test command parse its arguments instead of forwarding them
to spread in raw form. This allows us to use consistent double-dashed
long options, and have better help messages.

Signed-off-by: Claudio Matsuoka claudio.matsuoka@canonical.com

Let the test command parse its arguments instead of forwarding them
to spread in raw form. This allows us to use consistent double-dashed
long options, and have better help messages.

Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
@cmatsuoka cmatsuoka requested review from lengau and mr-cal May 17, 2024 22:46
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
@cmatsuoka cmatsuoka requested a review from mattculler May 23, 2024 01:00
@cmatsuoka cmatsuoka added this pull request to the merge queue May 24, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 24, 2024
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
@cmatsuoka cmatsuoka enabled auto-merge May 24, 2024 21:54
@cmatsuoka cmatsuoka added this pull request to the merge queue May 25, 2024
Merged via the queue into main with commit 58d3666 May 25, 2024
16 checks passed
@cmatsuoka cmatsuoka deleted the work/fix-test-cli-arguments branch May 25, 2024 16:59
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 this pull request may close these issues.

3 participants