takasshii / APIPracticeApp

GithubAPIからランキング一覧を取得して表示するアプリです。
0 stars 1 forks source link

Develop/changeAppColor #18

Closed shirataki707 closed 2 years ago

shirataki707 commented 2 years ago

変更の概要

なぜこの変更をするのか

やったこと

変更内容

before after
before(night) after(night)

影響範囲

どうやるのか

課題

備考

shirataki707 commented 2 years ago

プルリクのテンプレートはきちんと表示されていました.ありがとうございます!

takasshii commented 2 years ago

変更ありがとうございます!プルリクも非常に分かりやすくて良い感じです!! nichijou2

takasshii commented 2 years ago

forkするとリモートにブランチが生えないんですね…(はじめて知りました。) AndroidStudio上から変更を見るにはcloneするしかないんでしたっけ…??

shirataki707 commented 2 years ago

自分のリモートにforkしてクローンしないで,直接takasshiiさんのリポジトリをクローンしなきゃいけなかった感じですか?

takasshii commented 2 years ago

一度そちらの方針でやってもらえると助かります🙇 お手数お掛けして申し訳ないです!!!

takasshii commented 2 years ago

このブランチはマージしちゃって大丈夫です!!

shirataki707 commented 2 years ago

直接takasshiiさんのリポジトリをクローンして,試しにdevelop/testみたいなブランチ作ってコメント追加程度の適当なコミットしてプッシュしようとしたら,権限がないということではじかれました.そちらで何かできそうですか?

shirataki707 commented 2 years ago

このブランチはマージしちゃって大丈夫です!!

今回の場合だとtakasshiiさんが僕の分をマージすると思うんですけど,その場合今進んでいる分のmasterの変更(昨日takasshiiさんがマージした分)をこのブランチにマージして再pushしたものを取り込んでもらうという流れですか? ただforkしてきたからかandroid studio側からtakasshiiさんの方の更新を取得できない状況ではありますが...直接クローンした場合はtakasshiiさんの更新が見れるようです.git周り無知で申し訳ないです!

takasshii commented 2 years ago

collaboratorに追加してみました、、招待のメール受けて再度試してみてもらえると…🙏

takasshii commented 2 years ago

いやいやこちらこそgit周り無知で申し訳ないです… 今回はコンフリクトが生じてないのでそのままマージしちゃって大丈夫だと思います!! (今回追加した差分がmasterに追加されるだけだと思います)

shirataki707 commented 2 years ago

collaboratorありがとうございます! 後で消しますが今develop/testっていうブランチ作ってpushできたので問題なさそうです. マージもさっきまで自分の方ではできなかったんですが,今できるようになりました.