-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
coverage: add tests for arguments normalization #30
Conversation
c59d0ea
to
7b7f466
Compare
CI is failing because it uncovered a bug: Assigning to Fixed in #25 |
a963d96
to
8ce43d1
Compare
@bcoe See my comment above: #30 (comment) #25 is actually fixing this bug. |
@demurgos mind rebasing and let's see if tests pass. |
b813cea
to
afc1466
Compare
# Why `foreground-child` has a complex signature but not all its variants are covered by tests. # What Add tests for the signatures not covered by the other tests.
b7fb289
to
75182bd
Compare
Why
foreground-child
has a complex signature but not all its variants are covered by tests.What
Add tests for the signatures not covered by the other tests.
Other
Blocked on #25 (to fix a bug discovered by this PR)