azproduction / lmd

LMD - JavaScript Module-Assembler for building better web applications :warning: Project is no longer supported :warning:
http://azproduction.ru/lmd/
MIT License
449 stars 27 forks source link

создаем LMD модуль #123

Closed entcor closed 11 years ago

entcor commented 11 years ago

если написать в модуле так:

(function (require, exports, module) { 

});

, то сборка не будет работать, т.к. этот кусок в сборке будет выглядеть так:

"somemodule.js": (function (require, exports, module) {
});
,

последние точка с запятой все портят !

azproduction commented 11 years ago

Хм хм это я когда-то чинил, но почему-то у тебя воспроизводится :/

Вот это норм

(function (require, exports, module) {

module.exports =  {
    // Which filter are we using?
    TodoFilter: '', // empty, active, completed

    // What is the enter key constant?
    ENTER_KEY: 13
};

});

Чистит эту точку с запятой вот такая функция

var removeTailSemicolons = function (code) {
    return code.replace(/\n*;\n*$/, '');
};

Кинь модуль полностью куда-нибудь

azproduction commented 11 years ago

А, кстати, почему ты пишешь обертку сам, а не используешь CommonJS стиль?

module.exports =  {
    // Which filter are we using?
    TodoFilter: '', // empty, active, completed

    // What is the enter key constant?
    ENTER_KEY: 13
};
entcor commented 11 years ago

А, кстати, почему ты пишешь обертку сам, а не используешь CommonJS стиль?

ну сначала я делал так .... оно и осталось ... как я понимаю - это не страшно.

А ошибка скорее всего вылезла из-за использования async !

Вопрос не в тему : Видел твою статью про jsdoc. ты ссылаешься на переход на dox. видел его но остановило: Dox no longer generates an opinionated structure or style for your docs, it simply gives you a JSON representation, allowing you to use markdown and JSDoc-style tags.

Чем лучше это в читабельный вид превратить !

Спасибо!

azproduction commented 11 years ago

Для меня это так же новость :/ Даже не знаю.

azproduction commented 11 years ago

Закрываю таск?

entcor commented 11 years ago

да