Dev

[ghost]

No description provided.

[airtonix]

this should be node_modules

Suggested change

	node_module
	node_modules

[airtonix]

Suggested change

	require('dotenv').config();
	const nconf = require('nconf');

[airtonix]

I would suggest using nconf over dotenv, it's far more "12 factor" and a better user experience.

Suggested change

	const apiKey = process.env.API_KEY;
	const privateKey = process.env.PRIVATE_KEY;
	const ENV_SEPARATOR = '__';
	const KEY = 'btcmarkets';
	const ENV_PATTERN = new RegExp(`/^${KEY.toUpperCase()}${ENV_SEPARATOR}/`);
	nconf
	.env({
	separator: ENV_SEPARATOR,
	match: ENV_PATTERN,
	lowerCase: true,
	parseValues: true,
	transform(obj) {
	obj.key.replace(ENV_PATTERN, '');
	return obj;
	},
	})
	.argv({
	apiKey: {
	describe: 'prefix of all new packages created',
	},
	privateKey: {
	describe: 'prefix of all new packages created',
	}
	});
	const apiKey = nconf.get('apiKey');
	const privateKey = nconf.get('privateKey');

Now the cli is used like this:

node client.js --apiKey=something --privateKey=another-thing

or

BTCMARKETS_API_KEY=something \
BTCMARKETS_API_PRIVATE=another-thing \
node client.js 

you'll also get :

node client.js --help