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

Add a defmethod hook to multimethods to do custom stuff when a new method is added #108

Open
camsaul opened this issue Aug 24, 2022 · 2 comments
Labels
enhancement New feature or request
Milestone

Comments

@camsaul
Copy link
Owner

camsaul commented Aug 24, 2022

One possibility is that multimethods that normally use compound dispatch values can throw an error if you add a dispatch value with the wrong number of args (except for :default).

@camsaul camsaul added enhancement New feature or request high-priority! more important than the other issues labels Aug 24, 2022
@camsaul
Copy link
Owner Author

camsaul commented Sep 1, 2022

#113 would solve the use case here. Once we have that in we might not need this anymore.

@camsaul camsaul removed the high-priority! more important than the other issues label Sep 1, 2022
@camsaul
Copy link
Owner Author

camsaul commented Sep 9, 2022

This COULD be done with a ,vanilla?) multimethod that dispatches off of the multimethod. But that doesn't work for programmatic ones. So this should probably be an option passed during creation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant