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

Have ibids in JSON use a readable path rather than an index #29

Closed
erights opened this issue Jun 3, 2019 · 1 comment
Closed

Have ibids in JSON use a readable path rather than an index #29

erights opened this issue Jun 3, 2019 · 1 comment
Assignees
@erights
Labels
marshal package: marshal SwingSet package: SwingSet

Comments

@erights
Copy link
Member

erights commented Jun 3, 2019

At https://github.com/Agoric/SwingSet/issues/30#issuecomment-498065550 @dckc asks:

Can you use a path rather than an index, a la cycle?

It looks like Crock updated cycle since dscape forked it.
https://github.com/douglascrockford/JSON-js/blob/master/cycle.js
is nicely small. So, perhaps. The resulting JSON text would certainly be more human readable and maintainable, which is the only point of continuing to use JSON.
@warner warner transferred this issue from Agoric/SwingSet Dec 1, 2019
@warner warner added the SwingSet package: SwingSet label Dec 1, 2019
dckc pushed a commit to dckc/agoric-sdk that referenced this issue Dec 5, 2019
@michaelfig
Merge pull request Agoric#29 from Agoric/27-rename-functions
5bc9430 
rename functions to something shorter.
dckc pushed a commit to dckc/agoric-sdk that referenced this issue Dec 5, 2019
@michaelfig
Merge pull request Agoric#29 from Agoric/dependabot/npm_and_yarn/inte…
29bfb81 
…gration-test/mixin-deep-1.3.2

build(deps): bump mixin-deep from 1.3.1 to 1.3.2 in /integration-test
@erights erights added the marshal package: marshal label Apr 19, 2020
@erights erights self-assigned this Feb 25, 2021
@erights
Copy link
Member Author

erights commented Feb 15, 2022

Ibids are long gone. Closing.

@erights erights closed this as completed Feb 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@erights erights

Labels
marshal package: marshal SwingSet package: SwingSet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@warner @erights