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

Arr should be an array of RESP objects #1

Open
hellerve opened this issue Jan 27, 2021 · 1 comment
Open

Arr should be an array of RESP objects #1

hellerve opened this issue Jan 27, 2021 · 1 comment
Assignees
@hellerve
Labels
blocked by upstream This issue is blocked by an upstream dependency. bug Something isn't working

Comments

@hellerve
Copy link
Member

Currently Arr, which represents RESP arrays, is an array of strings. In truth, it should be an array of RESP objects to facilitate nesting and such; currently this does not work however, because we cannot define recursive datatypes (carp-lang/Carp#623).

When this is fixed, this library will be fixed also.

Cheers
@hellerve hellerve added bug Something isn't working blocked by upstream This issue is blocked by an upstream dependency. labels Jan 27, 2021
@hellerve hellerve self-assigned this Jan 27, 2021
@hellerve
Copy link
Member Author

Working on it in the recursive branch, using the work in carp-lang/Carp#1337.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hellerve hellerve

Labels
blocked by upstream This issue is blocked by an upstream dependency. bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hellerve