Add test for duplicate lint entries about eslint-config-hapi HOT 13 CLOSED

how would you do this, the moment you require it it will just take the last occurrence of the key

from eslint-config-hapi.

some low level read line by line could work
http://stackoverflow.com/questions/6156501/read-a-file-one-line-at-a-time-in-node-js/32599033#32599033

from eslint-config-hapi.

The config is just a couple of objects. Just loop over the keys and maintain a list of already seen keys.

from eslint-config-hapi.

the problem, as stated in first comment, is that when requiring, js will remove dupe keys and just assign the last value to it, so you will never have dupe keys

from eslint-config-hapi.

Oh, duh you're right. Yea, you would have to do it at the source code level.

from eslint-config-hapi.

👍

from eslint-config-hapi.

On second thought, this might be overkill. Closing.

from eslint-config-hapi.

yeah maybe, it is probably more sane to just go through it once and then keep it in mind for future updates

from eslint-config-hapi.

Conveniently, this is taken care of now, as of 7e71173

from eslint-config-hapi.

I feel dumb now haha also are you linting a linting config? meta

from eslint-config-hapi.

Yes. lintception

from eslint-config-hapi.

and belly button uses lab which uses this === infinite dependency :O

from eslint-config-hapi.

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.

from eslint-config-hapi.