Project

General

Profile

提案 #1269

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

Added by cob odo 10 months ago. Updated 9 months ago.

Status:
終了
Priority:
通常
Assignee:
-
Target version:
プラグイン名:
ブランチ:
topic/1269-handle-mobile-twitter-com

Description

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


Files

Associated revisions

Revision c21d340d (diff)
Added by cob odo 9 months ago

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

History

#1

Updated by Izumi Tsutsui 10 months ago

  • Tracker changed from 機能 to 提案
  • Status changed from 新規 to 分類待ち
  • Start date deleted (2018-06-15)

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

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

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

Updated by cob odo 10 months ago

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

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

ということなので、

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

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

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

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

#3

Updated by Izumi Tsutsui 10 months ago

cob odo さんは書きました:

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

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

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

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

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

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

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

#4

Updated by cob odo 9 months ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • ブランチ set to topic/1269-handle-mobile-twitter-com

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

#5

Updated by cob odo 9 months ago

  • Status changed from レビュー待ち to マージ待ち

動作かくにん! よかった

#6

Updated by cob odo 9 months ago

masterにマージしました

#7

Updated by cob odo 9 months ago

  • Status changed from マージ待ち to 終了

Also available in: Atom PDF