Avoid callback hell about aegir HOT 8 CLOSED

This one could be just part of the CR process. Make sure it is always easy to read.

from aegir.

cc @diasdavid @noffle @nginnever @RichardLitt

from aegir.

I think that enforcing max-depth makes sense. Refactoring is easier. Adding it to the ESLint and then having a note about why in the js-contributing.md sounds good to me. Shall I?

from aegir.

Make sure it is always easy to read.

CR failed so far catching this consistently if I look at some parts of the js-ipfs-* code base, so while that should be a goal I feel we should enforce it using the linter.

Adding it to the ESLint and then having a note about why in the js-contributing.md sounds good to me. Shall I?

If everyone is happy, yes please :)

from aegir.

What if we make the max being 4, because you know, with 2 spaces tabs, that is just 8 spaces total, not that bad :D

from aegir.

@diasdavid 4 sounds 👍 to me

from aegir.

let's do it :)

from aegir.

Ahhh Ok! Thanks @dignifiedquire

from aegir.