Xo: `optionsManager.buildConfig` only includes some options

Created on 9 Apr 2016  路  6Comments  路  Source: xojs/xo

It's filtering out many options, like cwd, filename, etc.

I had to work around it here, but would like to know the reason for it being like that. @jamestalmage ?

Probably one of the reasons for https://github.com/sindresorhus/atom-linter-xo/issues/19

bug question

Most helpful comment

Why on earth did I do that?

All 6 comments

I have strong suspicions it is related to https://github.com/sindresorhus/deep-assign/issues/6

@jamestalmage I don't think so. I explicitly use v1 of deep-assign, not v2 as mentioned there: https://github.com/sindresorhus/xo/blob/9fce4fb1a59abb28a47b2bbbc0ce3d922eb163ab/package.json#L67

Hmm.

Looking at the code, I don't see any reason cwd and filename wouldn't be copied.

Is it only affecting lintText?

Looking at the code, I don't see any reason cwd and filename wouldn't be copied.

You explicitly pluck out only some of the options here: https://github.com/sindresorhus/xo/blob/9fce4fb1a59abb28a47b2bbbc0ce3d922eb163ab/options-manager.js#L78-L82

Is it only affecting lintText?

Not sure. That's where I noticed it.

Why on earth did I do that?

Fixed since 3621ac5923afef15fd8f051518a688c8d6d1cb30

Was this page helpful?
0 / 5 - 0 ratings

Related issues

sindresorhus picture sindresorhus  路  5Comments

haskellcamargo picture haskellcamargo  路  3Comments

SamVerschueren picture SamVerschueren  路  4Comments

dawsbot picture dawsbot  路  5Comments

vadimdemedes picture vadimdemedes  路  3Comments