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.

Sign up for GitHub

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

Jump to bottom

Record multimethod ^:name metadata (for defmulti) and include it in "No matching method" errors #68

Closed
camsaul opened this issue Jun 11, 2021 · 2 comments
Closed

Record multimethod ^:name metadata (for defmulti) and include it in "No matching method" errors #68

camsaul opened this issue Jun 11, 2021 · 2 comments
Labels
enhancement New feature or request high-priority! more important than the other issues
Milestone
0.12.2

Comments

@camsaul
Copy link
Owner

camsaul commented Jun 11, 2021

Seems like it would be extremely helpful to know which multimethod a No matching method for dispatch value error applies to.
@camsaul camsaul added enhancement New feature or request high-priority! more important than the other issues labels Jun 11, 2021
@camsaul
Copy link
Owner Author

camsaul commented Jun 11, 2021

We should consider whether we should return an ExceptionInfo instead of an UnsupportedOperationException, and whether it should include info such as the original args and which dispatch values has primary methods

@jeff303 jeff303 mentioned this issue Oct 1, 2021
Merged
@camsaul camsaul added this to the 0.12.2 milestone Oct 20, 2021
@camsaul
Copy link
Owner Author

camsaul commented Oct 20, 2021

Implemented by #80

@camsaul camsaul closed this as completed Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request high-priority! more important than the other issues
Projects
None yet
Milestone
0.12.2
Development

No branches or pull requests
1 participant
@camsaul