Skip to content

Bump jsonrpclib to 0.2.0; make all jsonRpcPayloads required#15

Closed
kubukoz wants to merge 4 commits into
mainfrom
make-payloads-required
Closed

Bump jsonrpclib to 0.2.0; make all jsonRpcPayloads required#15
kubukoz wants to merge 4 commits into
mainfrom
make-payloads-required

Conversation

@kubukoz

@kubukoz kubukoz commented Mar 8, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@kubukoz

kubukoz commented Apr 11, 2026

Copy link
Copy Markdown
Contributor Author

@copilot update the test sources to match the new behavior

Copilot AI commented Apr 11, 2026

Copy link
Copy Markdown

@copilot update the test sources to match the new behavior

Done in 1d35996. Added @required to the result field in the expected output of the two relevant test cases ("request with input and output compiles" and "base type null is filtered from union variants").

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • s01.oss.sonatype.org
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java -XX:+HeapDumpOnOutOfMemoryError -cp /home/REDACTED/.cache/mill/download/0.12.14 mill.REDACTED.MillServerMain /home/REDACTED/work/lsp-smithy/lsp-smithy/out/mill-server/c161aec1ac85104478b08ab0b3cb8f70892b4a37-1 (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Comment thread lspSmithy/src/org/scala/abusers/lspsmithy/SmithyConverter.scala Outdated
@kubukoz kubukoz force-pushed the make-payloads-required branch 2 times, most recently from e8b3641 to c59e6ff Compare May 5, 2026 01:20
@kubukoz kubukoz marked this pull request as ready for review May 5, 2026 01:20
@kubukoz kubukoz force-pushed the make-payloads-required branch from c59e6ff to abd16e9 Compare May 5, 2026 01:21
Comment thread build.sc Outdated
@kubukoz kubukoz requested a review from ghostbuster91 May 5, 2026 10:13
@kubukoz kubukoz force-pushed the make-payloads-required branch from 6143a85 to 3c284ee Compare June 3, 2026 13:31
@kubukoz kubukoz changed the title Make all jsonRpcPayloads required Bump jsonrpclib to 0.2.0; make all jsonRpcPayloads required Jun 3, 2026
@kubukoz kubukoz closed this Jun 3, 2026
@kubukoz kubukoz deleted the make-payloads-required branch June 3, 2026 13:48
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