jacob-meacham / galactic-ansible

:alien: The Galactic Ansible
Apache License 2.0
0 stars 0 forks source link

Get up to speed on code #1

Open jacob-meacham opened 7 years ago

jacob-meacham commented 7 years ago

@nsimonson I'm not wed to any of this code at all - it was written a while ago.

We should also probably spend an hour modernizing the infrastructure here. In particular, now that modern node supports most of es6, we can rip out a lot of the babel transforms. I'm happy to do this

nsimonson commented 7 years ago

Can't get some of the scripts to run. dependency-check package.json --entry src/ is failing for me.

Console output

``` /Users/Niko/Source/galactic-ansible/node_modules/acorn/dist/acorn.js:2490 throw err ^ SyntaxError: 'import' and 'export' may appear only with 'sourceType: module' (1:0) at Parser.pp$4.raise (/Users/Niko/Source/galactic-ansible/node_modules/acorn/dist/acorn.js:2488:13) at Parser.pp$1.parseStatement (/Users/Niko/Source/galactic-ansible/node_modules/acorn/dist/acorn.js:762:14) at Parser.pp$1.parseTopLevel (/Users/Niko/Source/galactic-ansible/node_modules/acorn/dist/acorn.js:672:23) at Parser.parse (/Users/Niko/Source/galactic-ansible/node_modules/acorn/dist/acorn.js:529:15) at Object.parse (/Users/Niko/Source/galactic-ansible/node_modules/acorn/dist/acorn.js:3378:37) at parse (/Users/Niko/Source/galactic-ansible/node_modules/detective/index.js:9:18) at Function.exports.find (/Users/Niko/Source/galactic-ansible/node_modules/detective/index.js:44:15) at module.exports (/Users/Niko/Source/galactic-ansible/node_modules/detective/index.js:23:20) at read (/Users/Niko/Source/galactic-ansible/node_modules/dependency-check/index.js:171:22) at tryToString (fs.js:455:3) ```

jacob-meacham commented 7 years ago

Stupidly, I didn't peg dependencies in this repo. I'll peg dependencies and then we can decide whether to update them

nsimonson commented 7 years ago

Sounds good. I can check in a yarn.lock file too which accomplishes something similar... if you're using yarn.

jacob-meacham commented 7 years ago

yeah, I'm not opposed to yarn, but explicitly pegging dependencies in package.json is still the right move, imo