Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OSS Gate Workshop: Monotaro: 2019-12-02: tcs1203: Jquery: Work log #1354

Closed
tcs1203 opened this issue Dec 2, 2019 · 7 comments
Closed

OSS Gate Workshop: Monotaro: 2019-12-02: tcs1203: Jquery: Work log #1354

tcs1203 opened this issue Dec 2, 2019 · 7 comments

Comments

@tcs1203
Copy link
Contributor

@tcs1203 tcs1203 commented Dec 2, 2019

This is a work log of a "OSS Gate workshop".
"OSS Gate workshop" is an activity to increase OSS developers.
Here's been discussed in Japanese. Thanks.

作業ログ作成時の説明

以下のテンプレートを埋めてタイトルに設定します。埋め方例はスクロールすると見えてきます。

OSS Gate Workshop: ${LOCATION}: ${YEAR}-${MONTH}-${DAY}: ${ACCOUNT_NAME}: ${OSS_NAME}: Work log

タイトル例↓:

OSS Gate Workshop: Tokyo: 2017-01-16: kou: Rabbit: Work log

OSS Gateワークショップ関連情報

  • スライド:ワークショップの進行に使っているスライドがあります。
  • チャット:OSS開発に関することならなんでも相談できます。ワークショップが終わった後もオンラインで相談しながら継続的にOSSの開発に参加しましょう!
  • シナリオ:ワークショップの目的・内容・進め方の詳細が書いています。
  • 過去のビギナーの作業ログ:他の人の作業ログから学べることがいろいろあるはずです。
@tcs1203

This comment has been minimized.

Copy link
Contributor Author

@tcs1203 tcs1203 commented Dec 2, 2019

インストールを始める。
・Node.js をインストール
 →LTS と Current どっちをインストールすればいいのか?そもそもこいつら何?
  →https://tweeeety.hateblo.jp/entry/2018/05/22/194312
  →LTS は長期サポートをしている。例えばUbuntuだったら、メジャーバージョンが2個あがったらLTSがリリースされる。
 →ドキュメントには最新のNode.jsとしか書かれていない。
  →LTSでよいのかCurrentでよいのかここからは読み解けない。(リンクのページに飛ぶと、バージョンは書かれている)
 →Current をインストール

・gitでbilud開始
 →git clone git://hogehoge のgit:// のところはデフォルト設定ではないので、URLに書き換えて実行
  →であれば、git clone URL とすべきでは
 →cloneは成功!
 →npm run build しようと思ったけどそもそも npmとはなんぞや?
  →node.js のコマンド
 →npm run build を実行したら command not found になった
  →git bash 再起動で解消
 →npmコマンド実行したらエラーになった!

@tcs1203

This comment has been minimized.

Copy link
Contributor Author

@tcs1203 tcs1203 commented Dec 2, 2019

・npmコマンド実行のエラーを解消する
 →エラーログを見ました→よくわからん
 →Issue を見てみる
  →それっぽいものがあって、その中でPR が作成されていた
 →↑のPRのブランチをcloneしてきてnpmコマンド再実行
 →成功!

@tcs1203

This comment has been minimized.

Copy link
Contributor Author

@tcs1203 tcs1203 commented Dec 2, 2019

フィードバックすること

「To build jQuery, you need to have the latest Node.js/npm and git 1.7 or later.」とあるが、Node.js はバージョンの指定がなく、どれが最新かわからない。
READAMEにある以下のリンクに遷移すると、LTSとCURRENTがでてくる。最新と言われているので、CURRENTが当てはまるなとなんとなくわかるが、LTSもサポートされているかがわからないので、READAMEでバージョンを明記するか、LTSについて言及してほしい。
https://nodejs.org/en/download/

cloneするところのコマンド「git clone git://github.com/jquery/jquery.git」はsshを使った方法であるが、このsshの設定はgit をインストールした時点ではデフォルトで設定されているわけではない。なので、ユーザによっては実行できないことがありうるので、「git clone https://github.com/jquery/jquery.git」も併記してあげるとよいと思う。

@tcs1203

This comment has been minimized.

Copy link
Contributor Author

@tcs1203 tcs1203 commented Dec 2, 2019


build に使用する Node.js/npm のバージョンは LTS もサポート対象ですか?
もしサポート対象ならどのバージョンからサポートされているか書いてください。
Does Node.js / npm used for build support LTS?
If supported, could you write supported version to document?

@tcs1203

This comment has been minimized.

Copy link
Contributor Author

@tcs1203 tcs1203 commented Dec 2, 2019


git clone git://github.com/jquery/jquery.git
以下の文章を追記しませんか?
git clone https://github.com/jquery/jquery.git
ssh接続が使えないひとのために書いてあると親切です。

git clone git://github.com/jquery/jquery.git
Could you like to add the following sentence?
git clone https://github.com/jquery/jquery.git
It is helpful to write it for people who cannot use ssh connection.

@tcs1203

This comment has been minimized.

Copy link
Contributor Author

@tcs1203 tcs1203 commented Dec 2, 2019

Issue を投稿した!
jquery/jquery#4555
jquery/jquery#4556

@oss-gate-issue-cleaner

This comment has been minimized.

Copy link

@oss-gate-issue-cleaner oss-gate-issue-cleaner bot commented Dec 3, 2019

おつかれさまでした!

後日引き続きワークショップがある場合

後日のワークショップで再開する際に、issueタイトルの日付を更新してreopenしてください!

ワークショップ終了後に続きがしたい場合

issueはクローズしますが、その後も作業を続けていただいて大丈夫です!

是非このissueを活用なさってください。

今後のイベントの案内

現在以下のイベントの開催を予定しています。

参加をお待ちしてます!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.