Skip to content

Conversation

@ghostbuster91
Copy link

No description provided.

import io.circe.DecodingFailure

private[langoustine] object jsonrpcIntegration:
object jsonrpcIntegration:
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thought: needed to make this public so that it can be used in the test.core.testkit class

): Option[t.In] =
for
js <- scala.util.Try(ujson.read(writeToArray(p))).toOption
js <- scala.util.Try(ujson.read(writeToArray(p.data))).toOption
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question: would it be possible for langoustine to switch from ujson to circe? That would help a lot with all these conversions.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@keynmol thoughts?

@ghostbuster91
Copy link
Author

tests passed but java.net.ProtocolException: Server redirected too many times (20)

@kubukoz
Copy link
Contributor

kubukoz commented May 28, 2025

tests passed but java.net.ProtocolException: Server redirected too many times (20)

nothing new here, think the repo isn't set up for snapshot publishing. Revert my change from ci.yml and it'll go away.

or figure out what's missing :)

@kubukoz
Copy link
Contributor

kubukoz commented May 28, 2025

tests passed but java.net.ProtocolException: Server redirected too many times (20)

nothing new here, think the repo isn't set up for snapshot publishing. Revert my change from ci.yml and it'll go away.

or figure out what's missing :)

I think the secrets are simply not visible in the job due to running in a fork. @keynmol grant us some restricted write access please? :)

@ghostbuster91 ghostbuster91 marked this pull request as ready for review July 11, 2025 16:03
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.

2 participants