Skip to content

Conversation

@minggangw
Copy link
Member

@minggangw minggangw commented May 23, 2025

#1030 changed to leverage BigInt to represent int64, which leads to the type error of the client example. This PR fixes the type error by updating the client example to use BigInt instead of number.

Fix: #1145

@coveralls
Copy link

Coverage Status

coverage: 84.763%. remained the same
when pulling 7239bee on minggangw:fix-exmaple-error
into 4405557 on RobotWebTools:develop.

@minggangw minggangw requested a review from Copilot May 23, 2025 04:32
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the client example to ensure a and b are passed as BigInt values, matching the service’s expected input types.

  • Converted a and b from Number to BigInt
  • Preserved random range logic while addressing the type error

@minggangw minggangw merged commit 8249941 into RobotWebTools:develop May 23, 2025
14 checks passed
minggangw added a commit that referenced this pull request May 23, 2025
#1030 changed to leverage `BigInt` to represent int64, which leads to the type error of the client example. This PR fixes the type error by updating the client example to use `BigInt` instead of `number`.

Fix: #1145
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.

Fix the type error of client example

2 participants