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 ref-resolve as RefLike protocol method #68
Add ref-resolve as RefLike protocol method #68
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a couple of suggestions/changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple more small suggestions, then I think we'll be ready to merge.
ef112eb
to
c306ebd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside from a couple of places where the indentation is non-standard, this looks fine. Can you fix the indentation then squash your commits?
c306ebd
to
87c22ab
Compare
Thanks! Can you update the commit message to:
Then it should be good to merge. |
This allows custom references to be added that resolve in different ways. For example, a RefMap type might act the same as a RefSet, but include the keys for each referenced value. See issue weavejester#63.
87c22ab
to
12abc93
Compare
These methods allow a RefLike implementation to determine which dependencies are optional for a reflike, and which dependencies are mandatory. Also renames `include-refsets?` to `optional-deps?`. Resolves: weavejester#77 See also: weavejester#63, weavejester#68
These methods allow a RefLike implementation to determine which dependencies are optional for a reflike, and which dependencies are mandatory. Also renames `include-refsets?` to `optional-deps?`. Resolves: weavejester#77 See also: weavejester#63, weavejester#68
These methods allow a RefLike implementation to determine which dependencies are optional for a reflike, and which dependencies are mandatory. Resolves: weavejester#77 See also: weavejester#63, weavejester#68
This allows implementing "RefThing"s beyond just Ref & RefSet that resolve
in a custom way. For example a RefMap would be like RefSet but would
resolve to a map
{k -> ref}
instead a refs set.Related to #63