search

steder / goose

Simple configuration driven SQL migration tool
MIT License
3 stars 0 forks source link

Why it gots to be yaml? #5

Open ldanielburr opened 12 years ago

ldanielburr commented 12 years ago

Seriously, why introduce a dependency on a yaml parser, when we already have json available in python2.6 and higher?

ldanielburr commented 12 years ago

Further, goose relies upon argparse, so you have to be using python 2.7 anyway, unless you want to make the argparse package a dependency as well, so I think json just makes better sense.

steder commented 12 years ago

I think YAML is simply less cumbersome than JSON, and of course JSON documents are a valid subset of YAML so users can choose to use JSON if they prefer.

I don't think that a non-stdlib dependency is really a problem. Installing goose is going to require you to deal with downloading dependencies (SqlAlchemy, MySQLdb, Psycopg2, etc).

I'd be cool with implementing a fallback that uses the json if yaml isn't available.