Added :no-externs option to deps-cljs task. Fixes #23.

cljsjs / boot-cljsjs

Helper tasks to aid the packaging of JS dependencies for Clojurescript projects

66 stars 22 forks source link

Added :no-externs option to deps-cljs task. Fixes #23. #24

Closed jeluard closed 9 years ago

jeluard commented 9 years ago

Introduced a new option no-externs to task deps-cljs which allows to package a library without externs files.

jeluard commented 9 years ago

Oops it shipped with an extra.. Well I guess if you accept #22 first it should be fine. Let me know.

martinklepsch commented 9 years ago

Don't know how I feel about using _ instead of a proper shorthand option, other than that it looks great. Your call :smile:

jeluard commented 9 years ago

Well it's mostly because I couldn't think of a proper letter. e looks weird in this case, n is already used. An alternative could be to use :externs (default to true). Then e would be acceptable. But no sure I like this either.

jeluard commented 9 years ago

Actually I just noticed boot uses uppercase letter for negation so I switched to E for no-externs.