Project

General

Profile

Actions
Copy link

バグ #1307

closed

Retriever::URIを削除する

Added by toshi_a 初音 almost 6 years ago. Updated over 5 years ago.

Status:
終了
Priority:
通常
Assignee:
-
Target version:
3.9
プラグイン名:
ブランチ:
topic/1307-remove-retriever-uri
クラッシュする:
No

Description

Retriever::URI はDiva gemが導入されてから Diva::URIに置き換えられている。

Actions
Copy link
 #1

Updated by toshi_a 初音 almost 6 years ago

互換性のためにaliasしようかと思ったけど、そもそもファイルごとロードされて無くて草

Actions
Copy link
 #2

Updated by toshi_a 初音 almost 6 years ago

  • Status changed from 実装待ち to レビュー待ち
  • Target version changed from 3.8 to 3.9
  • ブランチ set to topic/1307-remove-retriever-uri

一応既存クラスが消えるので3.8では止めておいて、developにmergeしましょう。
削除コミットを行ったので誰か確認お願い

Actions
Copy link
 #3

Updated by toshi_a 初音 almost 6 years ago

  • Assignee deleted (toshi_a 初音)
Actions
Copy link
 #4

Updated by Izumi Tsutsui over 5 years ago

モデレータでも 担当者じゃないと レビュー待ち→マージ待ち に変えられない、
というのが意図通りかわかりませんが、とりあえず core/lib/retriever/uri.rb を
消しても普段遣いマシンで特に問題なく動いているので、
マージしてしまっていいんじゃないでしょうか。

Actions
Copy link
 #5

Updated by toshi_a 初音 over 5 years ago

  • Status changed from レビュー待ち to マージ待ち
Actions
Copy link
 #6

Updated by toshi_a 初音 over 5 years ago

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

merged.

Actions
Copy link

Also available in: Atom PDF