プロジェクト

全般

プロフィール

提案 #1269

mobile.twitter.comで始まるtweetのperma_linkの引用に対応する

cob odo約1ヶ月前に追加. 2日前に更新.

ステータス:
終了
優先度:
通常
担当者:
-
対象バージョン:
プラグイン名:
ブランチ:
topic/1269-handle-mobile-twitter-com

説明

というパッチを書きました

関係しているリビジョン

リビジョン c21d340d (差分)
cob odo2日前に追加

mobile.twitter.comで始まるtweetのperma_link引用に対応する refs #1269

履歴

#1 Izumi Tsutsui19日前に更新

  • トラッカー機能 から 提案 に変更
  • ステータス新規 から 分類待ち に変更
  • 開始日 を削除 (2018-06-15)

トラッカー「提案」のテストも兼ねて変更してみます。

これはパッチの内容的にそのまま「パッチ適用待ち」にしてテストするだけのように見えますが、
このあたりのGOの判断を誰がどうするかがワークフローとして議論必要?

  • コミッター+モデレータのメンバーのうち少なくとも 2人がOKと言ったら進める?
  • slack や mastodon とかでとしぁさんと会話して「いいよ」だったら進めるとか?
  • 最終的に「パッチ適用待ち」にするときにコミッタの判断が入るなら最初は緩くてもいい?

#2 cob odo17日前に更新

提案チケットの進め方によれば、

ステータス「分類待ち」の場合
この場合、モデレータがその提案が妥当なものなのか判断し返信する。
妥当でないならばその理由を説明し、ステータスを「却下」に変更する。
モデレータがそれを取り込むべきと判断すれば、以下の条件に従って次に進める。

ということなので、

このあたりのGOの判断を誰がどうするか

は、「独断で(もちろん理由は挙げるとして)GO/NO-GOを判断できる権限がモデレータに与えられている」と解釈するのかなーと思いました。
そうは言っても判断が微妙な提案はあると思いますし、迷ったら適宜相談という形でいいのではないかと個人的には思います。

「提案」フローについては、単に「こういう機能がほしい」という提案が(非コミッタ・非プログラマ)から出た場合、モデレータがGOの判断をしてもまだパッチは無いので「パッチ適用待ち」にはできなそうだなとは思いました。

チケットの内容と全然違う話になってしまいました……

#3 Izumi Tsutsui17日前に更新

cob odo さんは書きました:

このあたりのGOの判断を誰がどうするか

は、「独断で(もちろん理由は挙げるとして)GO/NO-GOを判断できる権限がモデレータに与えられている」と解釈するのかなーと思いました。
そうは言っても判断が微妙な提案はあると思いますし、迷ったら適宜相談という形でいいのではないかと個人的には思います。

仕様としては歓迎するが、実装(コード)はわからん、という場合は迷うんですよね……。(非プログラマのつぶやき)

公開API変更のような互換性で禍根を残すようなものは別として、「やってみてイマイチだったら元に戻せばいい」という程度の変更なら
あまり気にせず「コミッタorモデレータのうちひとりがOKと判断すれば進める」というゆるいルールでよいような気はします。

「提案」フローについては、単に「こういう機能がほしい」という提案が(非コミッタ・非プログラマ)から出た場合、モデレータがGOの判断をしてもまだパッチは無いので「パッチ適用待ち」にはできなそうだなとは思いました。

「提案」フローの分類待ちの説明がまだ書きかけという感じですが、ステータスとしては「実装待ち」もあるので、
パッチがない提案の場合はそちらになると思います。

で、このチケットは「問題があったら戻せばいい」という類の変更と思うので、パッチ適用待ちにしてしまいますね。
軽く試したところでは動作も問題ないと思います。

#4 cob odo2日前に更新

  • ステータスパッチ適用待ち から レビュー待ち に変更
  • ブランチtopic/1269-handle-mobile-twitter-com にセット

ブランチ topic/1269-handle-mobile-twitter-com にpushしました。

#5 cob odo2日前に更新

  • ステータスレビュー待ち から マージ待ち に変更

動作かくにん! よかった

#6 cob odo2日前に更新

masterにマージしました

#7 cob odo2日前に更新

  • ステータスマージ待ち から 終了 に変更

他の形式にエクスポート: Atom PDF