ykws / OneTimePasswordExample

OneTimePassword Example
1 stars 1 forks source link

iOS TOTP OSS 仕様準拠の確認 #1

Closed NMai-source closed 3 years ago

NMai-source commented 3 years ago

設定画面の追加

ykws commented 3 years ago

@NMai-source バッジの意図についてはこちらを参考にして付けています https://qiita.com/iganin/items/aee297eade84849cc9cd

ラベル 意味
must-badge 対応必須 この対応がされていないとマージを認められない
imo-badge in my opinion 自分ならこう実装するけど、どう?
ask-badge 実装意図の確認
nits-badge 細かい指摘(コードを整える、不要な改行など)
suggestion-badge 提案、このようにしたらどうか( imo との違いは客観的に推奨されている実装の提案)
good-badge 良い点
memo-badge 次の段階に進むためのポイント
memo-badge コード理解のためのメモとしてのコメント
ykws commented 3 years ago

@NMai-source 作成が後手に回ってしまったのですが、以下の Pull Request Template に合わせて説明欄の変更をお願いします。

https://github.com/ykws/OneTimePasswordExample/blob/main/.github/pull_request_template.md

ykws commented 3 years ago

@NMai-source #3 設定画面の追加までをこの Pull Request とできるように branch を分けてもらえますか?

アルゴリズムの選択などは #2 の Issue 対応として、この Pull Request とは別で作成してください。

ykws commented 3 years ago

@NMai-source main に CI を導入したのでこの Pull Request に main を merge or rebase してみてください。

コミットがきれいになるので rebase がおすすめです。 以下、一例です。

手順の例

  1. リモートリポジトリに Pull Request 先のこのリポジトリを追加する(名前は何でも良いですが upstream が慣習です)
  2. リモートリポジトリの内容を取り込みます
  3. 今の branch に対してリモートの main の変更を取り込んで一本のコミットログに整形します
  4. 3の手順に変わって単に merge しても良いですが、マージコミットが発生してしまうのでコミットログが読みにくくなります

コマンドの例

git remote add upstream git@github.com:ykws/OneTimePasswordExample.git
git fetch upstream
git rebase upstream/main
ykws commented 3 years ago

10 と重複するのでこちらは閉じます。