Closed kohichi000000 closed 8 years ago
@kohichi000000
i2c.jsの Polyfill (worker.js
とi2c.js
)が入っちゃっているようです。
(正しくはbower install
で導入する?)
.gitignoreファイルを追加して対象外となるようにした方が良いかもしれません。
コメントありがとうございます。 そういえば入れたままにしてしまいました。 よろしければ.gitignoreを追記いただければ...
あとで直しておきます~
.gitignoreの内容は上のリンクと同じ内容で大丈夫ですー
そういえば、コーディング規約的なのはどうしましょ? Mozilla のに合わせます?? https://developer.mozilla.org/ja/docs/Mozilla/Developer_Guide/Mozilla_Coding_Style_Guide これでよければ、これに合わせてレビューしちゃいます。
@dadaa 確かにコーディング規約はあった方が良いかもしれませんね。 今回の話だけに限らなそうなので、別途issuesでも議題にあげます?
@MSakamaki あ、そうですね!今日話しちゃいましょう
お、実験台になってよかったw
規約まではちゃんと見れてないですが、変えたつもりです
i2c.jsの Polyfill (worker.js
とi2c.js
)がまだ入ってるようなので以下のコマンドで取り除いちゃってください。
examples/i2c-VEML6070/
フォルダ内で以下を実行
# 以下のコマンドでgit管理から除外します。
git rm -fr bower_components
git rm js/webi2c.js js/worker.js
examples/i2c-VEML6070/
フォルダ内で以下のコマンドで.gitignore
ファイルを作りgit管理の対象外にする。
echo "bower_components" > .gitignore
これでできていますかね? (意図的にコマンドでやらずにweb上だけでやってみているので、うまくいっているか・・・)
. gitignore
の件はOKです!
あとはbower_components/
フォルダとjs/webi2c.js
, js/worker.js
ファイルを削除してもらえれば。
js/webi2c.js
, js/worker.js
は消えてませんかね?
js/webi2c.js
, js/worker.js
ファイルは消えてるの確認しました、すいません;
とりあえずweb上で全部やってみました。 Webからできれば、どのOSの人でも障壁なくできるってことですからね。
内容確認しました、自分はOKです。
@kohichi000000 fabbleにあるリンク http://fabble.cc/kohichi000000/chirimen-veml6070 を
README.md
ファイルに説明とfabbleへのリンク追加。i2c-VEML6070/
フォルダのREADME.md
ファイルにi2c-SRF02のREADME.mdと同じようにfabbleへのリンクを追加上記終わればマージ しちゃって大丈夫だと思います。
@MSakamaki readme編集しました~。
ついでにこんなの書いておきました。 素人の私でもわかる「githubにおいてWebブラウザだけでプルリクする方法」
@kohichi000000 おお!いいまとめですね。 内容も問題ないのでマージしちゃいます。
UVセンサの接続例を作りました