Remove squeryl and JSON scala combinators parser - Githubissues

lift / framework

Lift Framework

http://liftweb.net

Apache License 2.0

1.27k stars 280 forks source link

Remove squeryl and JSON scala combinators parser #2026

Closed fanf closed 3 days ago

fanf commented 3 days ago

Remove dependencies that are not used anymore but still referenced, causing problems for futur upgrades:

squeryl is a dead project for years now, and it causes import problem in intellij. Since persistance module is deleted, there's no reason to keep it,
JSON in scala-combinator was removed post-1.1 branch as a "sample, confused for a real parsed" and should not be used. It will make impossible update to newer version of the llib. See: https://github.com/scala/scala-parser-combinators/issues/99

The changes are minimal and all tests pass.

pabloazul commented 3 days ago

Looks reasonable to me. I built it with sbt 1.6.2 and java 11.0.2.