bem / bem-core

BEM Core Library
https://ru.bem.info/technologies/classic/i-bem/
Other
275 stars 95 forks source link

Normalization deps style in modules #1602

Closed belozer closed 6 years ago

belozer commented 6 years ago

resolve #1600

1 dep

modules.define('module', ['dep1'], function(provide, dep1, module) {
// …
});

2+ deps

modules.define('module', [
    'dep1',
    'dep1'
], function(provide, 
    dep1,
    dep2
) {
// …
});
belozer commented 6 years ago

@veged

На данный момент просто сделал по правилу описанному выше (1 и 2+). Но возможно стоит использовать эти форматы оформления исходя из длинны названий модулей.

belozer commented 6 years ago

@veged так то готово. Но чёт CI ругается npm run test локально нормально проходит....

tadatuta commented 6 years ago

судя по всему по косвенным зависимостям приехал код на ES6, который взорвался в нашем тестовом окружении. я починю, как найдется минутка.

tadatuta commented 6 years ago

@belozer я поправил сборку в v4, можешь, пожалуйста, отребейзить PR?

belozer commented 6 years ago

@tadatuta отребейзил по v4

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at 83.66% when pulling f7d16157cde765049ab1bde8c4aeea90c912c012 on belozer:issues/1600@v4 into 116a8f2192f7fe9915efe4402c44383e4e1062a1 on bem:v4.