-
Notifications
You must be signed in to change notification settings - Fork 2
Manifest: should JSON output for devices be a dict or an array? #22
Copy link
Copy link
Open
Labels
bugSomething isn't workingSomething isn't workingenhancementNew feature or requestNew feature or requestquestionFurther information is requestedFurther information is requested
Description
Currently, when outputing the devices:
$ ledger-manifest ledger_app.toml -od
devices:
0. stax
1. nanos+
2. nanos
3. nanox
$ ledger-manifest ledger_app.toml -j -od
{"devices": ["nanos+", "stax", "nanox", "nanos"]}
... which prevents to directly use ledger-manifest in shell scripts (including GitHub actions). One needs to:
compatible_devices="$(ledger-manifest --output-devices "$APP_MANIFEST" -j | jq -rc '.devices')";
What should be the proper behavior?
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingenhancementNew feature or requestNew feature or requestquestionFurther information is requestedFurther information is requested