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

drop util.parsing.json #221

Closed
SethTisue opened this issue Jun 20, 2019 · 3 comments · Fixed by #227
Closed

drop util.parsing.json #221

SethTisue opened this issue Jun 20, 2019 · 3 comments · Fixed by #227

Comments

@SethTisue
Copy link
Member

@SethTisue SethTisue commented Jun 20, 2019

it would remain on 1.1.x, we'd drop it for 1.2.x

it's been deprecated for quite a while now

@Philippus wdyt?

part of the reason I'm interested is that it would make it easier to add a Dotty cross-build

@Philippus

This comment has been minimized.

Copy link
Member

@Philippus Philippus commented Jun 21, 2019

I think in the "Programming in Scala"-book you build a json parser yourself and not use this one right? In that case I think it's ok to drop it.

@SethTisue

This comment has been minimized.

Copy link
Member Author

@SethTisue SethTisue commented Jun 21, 2019

@bvenners wdyt?

SethTisue added a commit to SethTisue/scala-parser-combinators that referenced this issue Jul 8, 2019
@SethTisue

This comment has been minimized.

Copy link
Member Author

@SethTisue SethTisue commented Jul 8, 2019

(I don't think we need to wait on Bill for this. I think you are right about what the book recommends, and even if you happen to be wrong, we should still do the removal IMO)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.