提案 #1269
closedmobile.twitter.comで始まるtweetのperma_linkの引用に対応する
Description
というパッチを書きました
Files
Updated by Izumi Tsutsui about 6 years ago
- Tracker changed from 機能 to 提案
- Status changed from 新規 to 分類待ち
- Start date deleted (
2018-06-15)
トラッカー「提案」のテストも兼ねて変更してみます。
これはパッチの内容的にそのまま「パッチ適用待ち」にしてテストするだけのように見えますが、
このあたりのGOの判断を誰がどうするかがワークフローとして議論必要?
- コミッター+モデレータのメンバーのうち少なくとも 2人がOKと言ったら進める?
- slack や mastodon とかでとしぁさんと会話して「いいよ」だったら進めるとか?
- 最終的に「パッチ適用待ち」にするときにコミッタの判断が入るなら最初は緩くてもいい?
Updated by cob odo about 6 years ago
提案チケットの進め方によれば、
ステータス「分類待ち」の場合
この場合、モデレータがその提案が妥当なものなのか判断し返信する。
妥当でないならばその理由を説明し、ステータスを「却下」に変更する。
モデレータがそれを取り込むべきと判断すれば、以下の条件に従って次に進める。
ということなので、
このあたりのGOの判断を誰がどうするか
は、「独断で(もちろん理由は挙げるとして)GO/NO-GOを判断できる権限がモデレータに与えられている」と解釈するのかなーと思いました。
そうは言っても判断が微妙な提案はあると思いますし、迷ったら適宜相談という形でいいのではないかと個人的には思います。
「提案」フローについては、単に「こういう機能がほしい」という提案が(非コミッタ・非プログラマ)から出た場合、モデレータがGOの判断をしてもまだパッチは無いので「パッチ適用待ち」にはできなそうだなとは思いました。
チケットの内容と全然違う話になってしまいました……
Updated by Izumi Tsutsui about 6 years ago
- File satoru_takeuchi-1012109727810011138.png satoru_takeuchi-1012109727810011138.png added
- Status changed from 分類待ち to パッチ適用待ち
cob odo さんは書きました:
このあたりのGOの判断を誰がどうするか
は、「独断で(もちろん理由は挙げるとして)GO/NO-GOを判断できる権限がモデレータに与えられている」と解釈するのかなーと思いました。
そうは言っても判断が微妙な提案はあると思いますし、迷ったら適宜相談という形でいいのではないかと個人的には思います。
仕様としては歓迎するが、実装(コード)はわからん、という場合は迷うんですよね……。(非プログラマのつぶやき)
公開API変更のような互換性で禍根を残すようなものは別として、「やってみてイマイチだったら元に戻せばいい」という程度の変更なら
あまり気にせず「コミッタorモデレータのうちひとりがOKと判断すれば進める」というゆるいルールでよいような気はします。
「提案」フローについては、単に「こういう機能がほしい」という提案が(非コミッタ・非プログラマ)から出た場合、モデレータがGOの判断をしてもまだパッチは無いので「パッチ適用待ち」にはできなそうだなとは思いました。
「提案」フローの分類待ちの説明がまだ書きかけという感じですが、ステータスとしては「実装待ち」もあるので、
パッチがない提案の場合はそちらになると思います。
で、このチケットは「問題があったら戻せばいい」という類の変更と思うので、パッチ適用待ちにしてしまいますね。
軽く試したところでは動作も問題ないと思います。
Updated by cob odo about 6 years ago
- Status changed from パッチ適用待ち to レビュー待ち
- ブランチ set to topic/1269-handle-mobile-twitter-com
ブランチ topic/1269-handle-mobile-twitter-com にpushしました。