caolan / caolan/nodeunit

missing uglify-js, make browser fails on sed

オープン
#111 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
1.9k
フォーク
359
PR マージ指標
30日以内にマージされた PR はありません

説明

## Expected
1. uglify-js should be a dev-dependency in npm `package.json`
2. `make browser` builds correctly
## Actual

Output as follows...

```
# super hacky build script for browser version!
# etc...
echo "nodeunit.run = reporter.run;" >> dist/browser/nodeunit.js
echo "return nodeunit; })();" >> dist/browser/nodeunit.js
sed -i "/\@REMOVE_LINE_FOR_BROWSER/d" dist/browser/nodeunit.js
sed: 1: "dist/browser/nodeunit.js": extra characters at the end of d command
make: *** [browser] Error 1
```

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

npm package.json と make browser ターゲットを読み、make browser を実行して sed の失敗を再現します。uglify-js が開発依存関係として宣言され、ブラウザービルドが報告されたエラーなしで完了すれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript, node.js
領域
build-system
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。