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

Make norm method signatures cleaner #49

Merged
merged 1 commit into from Aug 17, 2018

Conversation

Projects
None yet
2 participants
@Tyil
Copy link

Tyil commented Aug 17, 2018

Using a where in signatures generally doesn't generate clear error messages. Besides, the signatures I updated don't really need most of what was there anyway, the $which was not even used, for instance. So, in an attempt to make it more concise and easier to understand, I updated these signatures.

I ran the tests using prove -e 'perl6 -Ilib' t, and all tests passed (on my machine).

Patrick Spek
@lichtkind

This comment has been minimized.

Copy link
Collaborator

lichtkind commented Aug 17, 2018

very good

@lichtkind lichtkind merged commit ee6db97 into pierre-vigier:master Aug 17, 2018

0 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment