search

harusys / newsh

Twitter トレンド、天気情報などを LINE に通知するアプリ（API）
MIT License
0 stars 0 forks source link

[Enhancement] 天気情報を取得 #42

Closed hrtk7623 closed 2 years ago

hrtk7623 commented 2 years ago

やったこと

・Gateway 　⇒Timerにget_weather()を追加 　⇒callbackのhandler.addにweather追加 ・Weather 　⇒weather取得API追加

harusys commented 2 years ago

Closes #13

hrtk7623 commented 2 years ago

@harusys 修正して再コミットしました！

harusys commented 2 years ago

コンフリクトが発生しているかつ、パイプラインがこけたままなので、 現時点の master をマージするようにしてください！

harusys commented 2 years ago

@harusys 修正して再コミットしました！

対応完了しているものは閉じて、weather 側みると実装で気になる点がちらほらあったので、指摘追加しました！ あと https://github.com/harusys/newsh/pull/42#issuecomment-1019274285 でもコメントしましたが、現時点の master マージしてコンフリクト解消お願いします！ （master マージするとflake8 のリンターチェック回りますが、そこでひっかかる部分もいくつかあったのでそこも解消しましょう）

hrtk7623 commented 2 years ago

指摘対応をしました（オブジェクトマッピングのところだけまだ未対応です）

コンフリクト解消のため，ローカル環境にてmasterをマージしたのですが，

・[作業ブランチ] 編集内容をコミットする マスタブランチに移動 ・[マスターブランチ] 『git merge [作業ブランチ名]』

で合っているでしょうか？（逆のような気もしており...）

harusys commented 2 years ago

コンフリクト解消のため，ローカル環境にてmasterをマージしたのですが， ・[作業ブランチ] 編集内容をコミットする マスタブランチに移動 ・[マスターブランチ] 『git merge [作業ブランチ名]』 で合っているでしょうか？（逆のような気もしており...）

逆です。。。作業ブランチ側で、git merge master（master を取り込む）です！

hrtk7623 commented 2 years ago

作業ブランチ側で、git merge master（master を取り込む）です！ 失礼しました．．！

・[作業ブランチ] 編集内容をコミットする マスタブランチに移動 ・[マスターブランチ] 『git merge [作業ブランチ名]』

のあとプッシュしようとして（もちろん）失敗になり， ・git pull でいまいまの最新をローカルの落とす ・[作業ブランチ] 編集内容をコミットする ・[作業ブランチ] 『git merge master』 を実施しました．

一度『[マスターブランチ] 『git merge [作業ブランチ名]』』を実施してしまっていることの修正等必要なのか，気にしています．．．

hrtk7623 commented 2 years ago

@harusys モデルに合わせた修正MRしましたので確認とマージお願いします！