Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

p2p2chのリファクタリングブランチのマージ #11

Merged
merged 37 commits into from Jun 11, 2015
Merged

p2p2chのリファクタリングブランチのマージ #11

merged 37 commits into from Jun 11, 2015

Conversation

hangingman
Copy link
Contributor

とりあえずビルドが通ったバージョンをpull request。
機能的には元のp2p2chに戻ってしまいました。

ケジメ案件

  • ブランチに変更貯めこみすぎ
  • Scalazの最新版使ってplay付属のテストがコンパイル出来ないのがひどい

ご参考

Scalazの歴史と概要

7.1.x系
・他のライブラリとの相性やバイナリ互換を考えて慎重を期すなら、まだ7.0.x使ったほうがいい
・そのあたり気にする必要ない場合や、7.1.xにしか入ってない機能を使いたい場合は、もう7.1.0使うべき
・バイナリ互換崩れないものは、7.0.xにバックポートされてる

ぶっちゃけScalazを使う必要があるのか(´・ω・`)?
Scalazを7.0.x系に直してやれば丸く収まるのではないかと思うが、コードの中身を見てないのでなんとも言えない。playが昔のspecs2を引っ張ってくるのもお節介すぎる。

Momijikawa and others added 30 commits March 19, 2015 19:38
* localdbパッケージへのDB処理の閉じ込め
* ThreadクラスやResponsesクラスによる処理単位のオブジェクト化
Conflicts:
	app/controllers/Application.scala
	app/controllers/Subject.scala
	app/views/subject.scala.html
	build.sbt
	conf/routes
	project/plugins.sbt
hiroyuki nagata added 7 commits June 8, 2015 23:51
 ・stagingに上がっているキャッシュの削除
 ・Scalaバージョンの修正
Viewのスクリプトを一時コメントアウト
ScalazVersion = "7.1.1"
ScalaVersion  = "2.10.4"
ScalazVersion = "7.1.1"
AkkaVersion   = "2.2.3"
Specs2Version = "3.1"

play-testが古いバージョンのSpecs2を引っ張ってくる不具合を修正
@windymelt
Copy link
Owner

ブランチに変更貯めこみすぎ

アイエエエ!!スミマセン!!

ぶっちゃけScalazを使う必要があるのか(´・ω・`)?

scalazをimportしている部分を捜してみた

DataFetchingBeacon.scala:L24

  • flatMapのsyntax sugarである>>=を使うためにimport

Subject.scala:L32など

  • L32: コレクションメソッドsequenceを使うためにimport
    • List[Option[A]] => Option[List[A]]
  • L37: シュガーシンタックス|>を使うためにimport(x |> fとするとf(x)と等価
  • 関数合成などなど
  • さすがに分かりにくかったのでoverlay-weaverブランチでscalazを排しています

Response.scala:L57など

  • 関数合成に二箇所使用
  • overlay-weaverブランチでmodels/Response.scalaに移動しています

正直scalazは不要です。ただsequenceメソッドを自前で実装しなければならなくなります。

@hangingman
Copy link
Contributor Author

あー、Subject.scalaが読めなかったのはそのためか…
Scalaz使うのはいいけどバージョンは後から落とせるかな?(コード的にはそうしたい)
これはコメントであって特に問題はないです

@windymelt
Copy link
Owner

尖った機能は使ってないのでたぶんバージョンは落とせます

windymelt pushed a commit that referenced this pull request Jun 11, 2015
p2p2chのリファクタリングブランチのマージ
@windymelt windymelt merged commit 8f658d7 into windymelt:master Jun 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants