-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Update protoquill #12187
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
Update protoquill #12187
Conversation
@deusaquilus I noticed that protoquill master is failing in our master. Is this the failure from #12179? |
Correct. I just commented all of that out though. Master build and tests are now all passing now. |
eef042c
to
07c08fc
Compare
3efb86b
to
6fe9cb5
Compare
@deusaquilus it seems the latest master of protoquill is timing out. Could you take this over and try to make protoquill/master work with the latest dotty nightly? |
Sure, I'll have a look. The issue probably is because it is depending on a SNAPSHOT version of quill-core-portable/quill-sql-portable that no-longer exists. I will publish regular versions of these things ASAP and try to fix this up by tomorrow. Thanks for the info! |
Updated Proto-Quill master. Removed scalatest dependency. It should be working now. |
6fe9cb5
to
1e39558
Compare
Updated to the latest master of protoquill |
Thanks! |
@nicolasstucki The stuff you did with Extractors.scala is absolutely amazing! I love it! @odersky Any chance you can vouch for this |
No description provided.