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

Release 1.0.0 #40

Closed
7 tasks done
mlegenhausen opened this issue Jul 31, 2014 · 3 comments
Closed
7 tasks done

Release 1.0.0 #40

mlegenhausen opened this issue Jul 31, 2014 · 3 comments
Assignees
Milestone

Comments

@mlegenhausen
Copy link
Owner

This is a meta task for organising some work to do:

  • Remove jsbeautify, this is a redundant functionality that can be replace with grunt-jsbeautifyer
  • Move all serializer dependent options to the options.serializer parameter
  • Add the 'use strict' to the default wrapper
  • Switch from double to single quotes when possible
  • Switch from the default JSON module to JSON5 or another serializer that support single quoted output.
  • Define behaviour on empty options
  • Update Readme
@mlegenhausen mlegenhausen added this to the Version 1.0.0 milestone Jul 31, 2014
@mlegenhausen mlegenhausen self-assigned this Jul 31, 2014
mlegenhausen pushed a commit that referenced this issue Jul 31, 2014
mlegenhausen pushed a commit that referenced this issue Jul 31, 2014
mlegenhausen pushed a commit that referenced this issue Jul 31, 2014
mlegenhausen pushed a commit that referenced this issue Jul 31, 2014
mlegenhausen pushed a commit that referenced this issue Jul 31, 2014
mlegenhausen pushed a commit that referenced this issue Jul 31, 2014
mlegenhausen pushed a commit that referenced this issue Jul 31, 2014
@mlegenhausen
Copy link
Owner Author

All changes can be found in the 1.0.0 branch.

Currently there is no serializer available that supports single quoted output. I hope JSON5 will jump on it. You can follow the progress at json5/json5#71.

@tlvince
Copy link
Contributor

tlvince commented Aug 16, 2014

@mlegenhausen, could you do a GitHub release/tag of this (since it's the latest stable in npm)? Thanks 😄

@mlegenhausen
Copy link
Owner Author

Tags added.

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

2 participants