Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

compile-json.sh is lame #22

Open
ozten opened this Issue Feb 7, 2013 · 1 comment

Comments

Projects
None yet
1 participant
Member

ozten commented Feb 7, 2013

Lame because

  • po2json.js isn't actually in node_modules/.bin
  • po2json.js file path is hard coded
  • this script assumes your at the top level of your directory

We should port this from bash to Node.js and resolve file paths to work from anywhere.

Member

ozten commented Feb 7, 2013

Low bar fixes are to rename the existing shell script to compile-json and fix the path error.

@ozten ozten added a commit that referenced this issue Feb 7, 2013

@ozten ozten Incremental fixes towards Issue #22 f13c9df

@ozten ozten added a commit that referenced this issue Feb 7, 2013

@ozten ozten Add executables to package.json
Rename compile-json.sh

Fixes #21 and helps Issue #22.
cc3556b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment