#1 Use JSON instead of YAML

Closed
opened 7 years ago by clara · 2 comments
clara commented 7 years ago

We should use JSON instead of YAML for configuration and other data. Rationale for this decision are as follows:

  • Either one would work just fine for this application
  • Python has a built-in JSON library
  • YAML is an external dependency
  • We currently have extra code so things don’t break if our YAML decoder isn’t available
  • Most importantly, I just like JSON
We should use JSON instead of YAML for configuration and other data. Rationale for this decision are as follows: - Either one would work just fine for this application - Python has a built-in JSON library - YAML is an external dependency - We currently have extra code so things don't break if our YAML decoder isn't available - Most importantly, I just like JSON
clara commented 7 years ago
Owner

YAML has been removed and replaced with JSON. There is, however, a problem with conflict resolution between command-line arguments and the configuration file. Once this is fixed, I will close the issue.

YAML has been removed and replaced with JSON. There is, however, a problem with conflict resolution between command-line arguments and the configuration file. Once this is fixed, I will close the issue.
clara commented 7 years ago
Owner

Configuration/argument behaviour fixed: see 87c0a1bfe9 for more details.

Configuration/argument behaviour fixed: see 87c0a1bfe93ffbe7eb67c41ea2b46fd5a7c1ab28 for more details.
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
Cancel
Save
There is no content yet.