Skip to content

Require @required trait on payloads#101

Merged
kubukoz merged 2 commits into
mainfrom
require-required-on-payload
Apr 18, 2026
Merged

Require @required trait on payloads#101
kubukoz merged 2 commits into
mainfrom
require-required-on-payload

Conversation

@kubukoz

@kubukoz kubukoz commented Apr 11, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@kubukoz

kubukoz commented Apr 12, 2026

Copy link
Copy Markdown
Contributor Author

@copilot fix tests by adding @required where necessary

Copilot AI commented Apr 12, 2026

Copy link
Copy Markdown
Contributor

@copilot fix tests by adding @required where necessary

Added @required to all members using @jsonRpcPayload in both tests (commit c597b3f). This covers the valid-usage test (input/output/error structures) and the nested-structure error test.

@kubukoz kubukoz marked this pull request as ready for review April 12, 2026 20:49
@kubukoz

kubukoz commented Apr 18, 2026

Copy link
Copy Markdown
Contributor Author

cc @keynmol / @ghostbuster91

@kubukoz kubukoz merged commit 288562f into main Apr 18, 2026
3 checks passed
@kubukoz kubukoz deleted the require-required-on-payload branch April 18, 2026 11:54
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.

3 participants