search

lukeed / taskr

A fast, concurrency-focused task automation tool.
MIT License
2.53k stars 74 forks source link

Fix formatting issues when process.env.HOME is undefined #219

Closed sylvainpolletvillard closed 7 years ago

sylvainpolletvillard commented 7 years ago

fix for https://github.com/flyjs/fly/issues/101

lukeed commented 7 years ago

Quick question, do you have process.env.USERPROFILE defined?

sylvainpolletvillard commented 7 years ago

Yes

image

Also I have process.env.HOMEPATH defined

lukeed commented 7 years ago

Ok. Do me a favor and try this instead:

var home = process.env[(process.platform === 'win32') ? 'USERPROFILE' : 'HOME']
var msg = pretty.render(e)
    .replace(/(\sFunction|\sObject)\./g, clor.blue('$1') + '.')
    .replace(/\((~?\/.*)\)/g, '(' + clor.gray('$1') + ')')
    .replace(/:([0-9]*):([0-9]*)/g, ' ' + clor.yellow('$1') + ':' + clor.yellow('$2'))
    .replace(new RegExp(home, 'g'), '~')
console.error(msg)
lukeed commented 7 years ago

@sylvainpolletvillard I'll take care of the lint errors. Thank you! 🙌