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

Added support for more kinds of object-name values #527

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

dented42
Copy link

This fixes the issue presented in #526. An object-name/c contract has been added and make-named-object-custom-write has been modified to accept the new kinds of names.

Updated documentation to reflect the new kinds of object names.
removed `object-name/c`, `make-named-object-custom-write` getter function can produce symbols or strings.
Removed `object-name/c`
@dented42
Copy link
Author

Editing files within GitHub is annoying, you'll want to squash these. I've removed object-name/c, since it was only used in one place. To keep track with the conversation regarding issue #526 I've removed numbers and byte strings from the types that make-named-object-custom-write accepts, leaving only symbols, strings, and paths.

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

Successfully merging this pull request may close these issues.

1 participant