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

yml support for configuration files #5

Closed
davidemoro opened this issue Jan 2, 2018 · 4 comments
Closed

yml support for configuration files #5

davidemoro opened this issue Jan 2, 2018 · 4 comments

Comments

@davidemoro
Copy link
Contributor

davidemoro commented Jan 2, 2018

Yaml will become the preferred serialization format instead of json (perhaps the unique serialization format if we decide to drop the json format).

Pro:

  • more user friendly
  • no brackets
  • less prone to errors (compared to json trailing commas for non technical users)
  • always parseable by default ({"property": $something} is not parseable and no easy way to implement a validator, where for $something we could have 1, "a string", null for example). property: $something can be validated

Cons:

  • a large amount of documentation (and third party plugins documentation) will need to be updated
  • in case in future releases we'll decide to drop json, no more backwards compatible (any strong opinion for maintaining json?)
  • play_json fixture soft deprecation and fixture rename
@davidemoro
Copy link
Contributor Author

not yet requested by any user, close

@davidemoro
Copy link
Contributor Author

Considered useful by users, reopen

@davidemoro davidemoro reopened this Feb 6, 2018
@davidemoro davidemoro changed the title add yaml support for configuration files yml support for configuration files Apr 26, 2018
@davidemoro
Copy link
Contributor Author

Feedback from users: deprecate json and use yaml.
play_json fixture deprecated and it will removed in version >=2.0.0

davidemoro pushed a commit that referenced this issue May 17, 2018
… yaml will be used and json will be no more supported. See #5
davidemoro added a commit that referenced this issue Jan 11, 2019
@davidemoro
Copy link
Contributor Author

Merged in features branch, waiting for new 2.0.0 release

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

No branches or pull requests

1 participant